Looking through recently merged commits, I noticed that changeset 80af8db446df changes <http://bitbucket.org/lindenlab/viewer-development/changeset/80af8db446df#chg-indra/llui/lluistring.h_newline113> LLUIString::mArgs into a pointer. This pointer gets initialized with a new object in one of the constructors <http://bitbucket.org/lindenlab/viewer-development/changeset/80af8db446df#chg-indra/llui/lluistring.cpp_newline41> (the other constructors set it to NULL), or lazily (i.e. when still NULL when an actual object is required) through LLUIString::getArgs() <http://bitbucket.org/lindenlab/viewer-development/changeset/80af8db446df#chg-indra/llui/lluistring.cpp_newline161>.
Though I can't seem to find where mArgs gets deleted again. Is it being leaked? As far as I can see, mArgs (which is private) doesn't get assigned any pointers from outside the LLUIString instance and is never passed out through any method, so it should be save to delete it in the destructor of LLUIString. Am I missing something obvious? Cheers, Boroondas
_______________________________________________ Policies and (un)subscribe information available here: http://wiki.secondlife.com/wiki/OpenSource-Dev Please read the policies before posting to keep unmoderated posting privileges
