[Libreoffice] [PUSHED] Re: [PATCH] Again cppcheck cleanliness

[Caolán McNamara]

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
again.

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.

C.