Date: prev next · Thread: first prev next last
2013 Archives by date, by thread · List index 

Hello,

Il 15/03/2013 09:30, julien2412 ha scritto:

Noel Grandin wrote

I would guess that someone meant to move line 742 inside the first "if"
statement, but only did half the job.


Hello Noël,

Perhaps I'm too sleepy this morning but I don't see the interest to
duplicate the "new" part. Any hint?
Looking at the switch a few lines before it looks like that nLeft and nRight can be both set at the set time so the code is right; the leak really depends on what is done with the pointer in SwLineLayout, which is the type of pCurr. So if SwLineLayout properly frees this pointers i'd just mark the issue a false positive. 

hth,
riccardo

Context

Privacy Policy | Impressum (Legal Info) | Copyright information: Unless otherwise specified, all text and images on this website are licensed under the Creative Commons Attribution-Share Alike 3.0 License. This does not include the source code of LibreOffice, which is licensed under the Mozilla Public License (MPLv2). "LibreOffice" and "The Document Foundation" are registered trademarks of their corresponding registered owners or are in actual use as trademarks in one or more countries. Their respective logos and icons are also subject to international copyright laws. Use thereof is explained in our trademark policy.