[Libreoffice] [PUSHED] Re: [PATCH] Again cppcheck cleanliness
caolanm at redhat.com
Fri Jan 7 03:53:07 PST 2011
On Fri, 2011-01-07 at 11:07 +0000, Caolán McNamara wrote:
> On Thu, 2011-01-06 at 20:36 +0100, Guillaume Poussel wrote:
> > Hi everybody,
> > A new patch which correct cppcheck warnings in base/.
> Looks good, pushed. Thanks for this.
Oops, I see now that cppcheck isn't run with sufficient -I flags to find
includes, so it wasn't able to see that pDesc is actually used by some
MACRO stuff which was hidden from it, so these were cppcheck false
positives, as are probably the "missing ctor" warnings.
I fixed up the pDesc ones again in a way to hide them from cppcheck
We probably should figure out how to sufficient -I flags to cppcheck to
get it to find the includes. This is probably a bit tricky unless we add
a make cppcheck or something to libreoffice itself or something.
More information about the LibreOffice