<br><br><div class="gmail_quote">2011/1/28 Michael Meeks <span dir="ltr"><<a href="mailto:michael.meeks@novell.com">michael.meeks@novell.com</a>></span><br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
Hi there,<br>
<div class="im"><br>
On Thu, 2011-01-27 at 20:55 +0100, Kenneth Venken wrote:<br>
> these patches remove some comments<br>
> from ./filters/binfilter/bf_svx/source/editeng/svx_editobj.cxx and<br>
<br>
</div> I've pushed these two.<br>
<br>
> fixes two memleaks.<br>
<br>
But this third one looks more problematic to me (at least on the<br>
surface) :-)<br>
<br>
We allocate pC and then store its pointer in aContents (cf.<br>
CreateAndInsertContent) - right ? surely they are freed by<br>
DeleteContents called from the destructor ?<br></blockquote><div>your right. I guess this is a cppcheck false positive. <br></div><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
<br>
Thanks though,<br>
<br>
Michael.<br>
<font color="#888888"><br>
--<br>
<a href="mailto:michael.meeks@novell.com">michael.meeks@novell.com</a> <><, Pseudo Engineer, itinerant idiot<br>
<br>
<br>
</font></blockquote></div><br>