I don't know if this is the right place, or if it's really an error, but I noticed something odd. While searching for a solution of decimal separator problem, I found in AdvGrid.pas unit the expression
if ... or (1 > 0) and the expression
if ... and (1 < 0) a couple of times.
This is not an error and it is intentional.
It is excluding code but still keep the code for later reference to the developer.