Do we really need to use strdup in the first place?
I think we could just update the WrappedText helper method to return std::string and then just invoke .c_str() off the return value for the ImGui methods, then the std::string will just be cleaned up when scope exits. Seems better than having to remember to call free (which there is a usage of WrappedText in the SaveEditor missing a free as well).

i'm down to switch to using std::string. also, can this point to macready?

closing in favor of #4144