Hi all,
    while looking at 
http://cgit.freedesktop.org/libreoffice/core/commit/?id=a5ab0e3a8b1cb7c06072229e1c4d956eb81fa002
 I noticed a really suspicious use of ! on the same line that was modified:

&& ! nMinNextEnd < nRow

Based on the 'n' prefix and the related usage of ! in conditionals on integers, 
I would say that that should be !(nMinNextEnd < nRow) or nMinNextEnd >= nRow

Sadly I don't have time to propose a patch in gerrit for review. Does anybody 
have a clue on what should go there, how to test the results and, if needed, 
patch the code?

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

Reply via email to