On Sun, 2011-05-08 at 15:09 +0200, Julien Nabet wrote:
> I found this with cppcheck :
> [vcl/unx/source/gdi/salbmp.cxx:795]: (style) Variable 'bFreePixmap' is 
> not assigned a value

        Ho hum - I beg to differ:

        bool bFreePixmap = false;
        if( bSuccess && (args[0] >>= bFreePixmap) && bFreePixmap )

        Surely we assign it a value with the >>= operator ? true - it is not a
particularly pleasant use of the operator - but hey - UNO is like that
(sexy huh?). Why have:

        any.isBoolean() && any.asBoolean()

        or something - when we can have some mind-mashed operator, and a
confusing temporary ;-)

        Or did I miss something ? presumably another cppcheck false-positive
that needs filing & fixing (?).

        Thanks,

                Michael.

-- 
 michael.me...@novell.com  <><, Pseudo Engineer, itinerant idiot


_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice

Reply via email to