<div dir="ltr"><div><div><div><div><div>Hi Maxim<br><br></div>From the FontList methods, it looks like it uses GetDevFontSize(), GetMapMode(), SetMapMode() and GetDevFontSizeCount() methods of<br></div>the object that is passed to FontList constructor. All these methods are only defined for OutputDevice class. So it would seem that passing the parent window should have <br></div>the same effect. Valgrind no more shows that memleak if we pass pBox->GetParent(). <br><br>Should I remove the newly added dispose() functions (for FontList etc..) as well ? Seems the above one liner change is enough to kill the memleak.<br><br></div>Thanks,<br></div>Dennis<br><div><div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Sep 1, 2015 at 2:56 AM, Maxim Monastirsky <span dir="ltr"><<a href="mailto:momonasmon@gmail.com" target="_blank">momonasmon@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<span><br>
<br>
On Mon, Aug 31, 2015 at 10:00 PM, Dennis Francis <<a href="mailto:dennisfrancis.in@gmail.com" target="_blank">dennisfrancis.in@gmail.com</a>> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
About passing SvxFontNameBox_Impl : I have not yet understood why we pass that, will check as soon as possible.<br>
</blockquote></span>
I will clarify what I meant. It seems to me that we pass the SvxFontNameBox_Impl only to get through it the font list from the underlying platform backend. In this case, we can do the same with other windows. For example, passing the ToolBox there (via pBox->GetParent()) seems to eliminate the memleak, while the font box seems (at least at first glance) to still work as expected. But you should take this with caution, given that I'm not really familiar with the font init stuff.<span><font color="#888888"><br>
<br>
Maxim<br>
<br>
</font></span></blockquote></div><br></div></div></div></div>