[Libreoffice] cppcheck : Same expression on both sides of '!='

Kohei Yoshida kyoshida at novell.com
Fri May 13 10:13:22 PDT 2011


On Fri, 2011-05-13 at 23:54 +0700, Korrawit Pruegsanusak wrote:
> Hello all,
> 
> Joop Kiefte wrote:
> > Maybe worth it to look those places up in the version control history...
> 
> I've annotated this file and found the added version since Sep 2010.
> Let's take a look from this link:
> http://opengrok.libreoffice.org/diff/calc/sc/source/core/tool/rangeutl.cxx?r2=%2Fcalc%2Fsc%2Fsource%2Fcore%2Ftool%2Frangeutl.cxx%40188130067b80960fd509078f2e23fc1636e694ab&r1=%2Fcalc%2Fsc%2Fsource%2Fcore%2Ftool%2Frangeutl.cxx%40f789f423c79cb68fdf253cbc0291139436e80ec8

The actual change that matters dates back older than that commit (back
in the go-oo times which means you need to dig up the history of the
actual patch file itself in the 'build' repository).

Anyway, in the latest master I've already made changes to make the code
the way it should be.

Kohei

-- 
Kohei Yoshida, LibreOffice hacker, Calc
<kyoshida at novell.com>



More information about the LibreOffice mailing list