On 09/08/2011 01:44 AM, Eike Rathke wrote:
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!
Korrawit, Eike, did you check the context to verify that it should
a.nChar1 == a.nChar2
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.)
LibreOffice mailing list