Hi Maxim Thanks for pointing out, I have taken out my call of disposeAndClear(). About passing SvxFontNameBox_Impl : I have not yet understood why we pass that, will check as soon as possible.
Hi Michael Thanks for the suggestions. I have attempted to do what you suggested in the latest patch uploaded at gerrit ( https://gerrit.libreoffice.org/#/c/18073/ ) I used opengrok to get a list of all usages of the token FontList, and tried to figure out which all objects instantiate FontList, there are places where I am not sure of the correctness of my patch. At least the unit tests did not fail and the original memleak is also gone :) But I am scared if introduced crashes in some untested work flows. Could you please review and suggest changes ? Thanks, Dennis On Mon, Aug 31, 2015 at 12:42 PM, Michael Meeks <michael.me...@collabora.com> wrote: > > And we of course need to dispose the FontList whereever it is > instantiated in a widget: eventually we'll have to hit a 'dispose' > method going up the tree. > > Is that feasible ? > > Thanks so much for chasing this one Dennis ! > > ATB, > > Michael.
_______________________________________________ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice