<div dir="ltr"><div><div><div><div><div><div><div><div>Hi Maxim<br><br></div>Thanks for pointing out, I have taken out my call of disposeAndClear().<br></div>About passing SvxFontNameBox_Impl : I have not yet understood why we pass that, will check as soon as possible.<br><br></div>Hi Michael<br><br></div>Thanks for the suggestions. I have attempted to do what you suggested in the latest patch uploaded at gerrit ( <a href="https://gerrit.libreoffice.org/#/c/18073/" target="_blank">https://gerrit.libreoffice.org/#/c/18073/</a> )<br></div>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<br></div>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<br></div><div>untested work flows.<br></div><div><br>Could you please review and suggest changes ?<br><br></div><div><br></div>Thanks,<br></div>Dennis<br><div><div><div><div><div><div><br><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Aug 31, 2015 at 12:42 PM, Michael Meeks <<a href="mailto:michael.meeks@collabora.com">michael.meeks@collabora.com</a>> wrote:<br>><br>>         And we of course need to dispose the FontList whereever it is<br>> instantiated in a widget: eventually we'll have to hit a 'dispose'<br>> method going up the tree.<br>><br>>         Is that feasible ?<br>><br>>         Thanks so much for chasing this one Dennis !<br>><br>>         ATB,<br>><br>>                 Michael.</div><br></div></div></div></div></div></div></div></div></div></div>