On 09/08/2011 01:44 AM, Eike Rathke wrote:
Hi Korrawit,
On Wednesday, 2011-09-07 22:01:09 +0700, Korrawit Pruegsanusak wrote:
Attached patch will clear the cppcheck warning for duplicate
expression on both sides of '<'.
Yup, good catch!
http://cgit.freedesktop.org/libreoffice/core/commit/?id=a58ed493e572fef2c503bd329e924bb062ba9c96
Thanks
Eike
Korrawit, Eike, did you check the context to verify that it should
really read
a.nChar1 == a.nChar2
instead of
a.nChar1 == b.nChar1
in the second term of
(a.nChar1 < b.nChar1) || ((a.nChar1 == a.nChar2) && (a.nChar2 < b.nChar2))
? Somehow, that expression still looks suspicious. (At least, it does
not implement a lexicographical less-than relation. Not sure whether it
should or not, however.)
-Stephan
_______________________________________________
LibreOffice mailing list
LibreOffice@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice
