Hi Julien, On Mon, 2011-05-16 at 19:08 +0200, Julien Nabet wrote: > Here a line I get in cppcheck (updated today) : > [source/core/layout/flycnt.cxx:1194] -> > [source/core/layout/flycnt.cxx:1194]: (style) Same expression on both > sides of '<'.
What fun :-) > I checked the file git history, it's there since the beginning. > while ( pDownFrm && ( ( nDown.nMain != LONG_MAX && > nDownLst < nDownLst So - this is the guy. sal_Bool operator<( const SwDistance& rTwo ) { return nMain < rTwo.nMain || ( So - we end up with: that equating to: nDownLst.nSub && nDownLst.nSub && nDownLst.nSub < nDownLst.nSub Which seems like it is always false ;-) so lets just axe that comparison if indeed we have this code in the first checkin (in 2000) ;-) Thanks ! Michael. -- michael.me...@novell.com <><, Pseudo Engineer, itinerant idiot _______________________________________________ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice