Possible Squirrel bug - fixed in r20162

[Attila7]

Can someone please very that the Squirrel statement
(13725 > -2147483648)
produces false?

In case it is verified, the culprit seems to be at line 199 of sqvm.cpp which reads:
_RET_SUCCEED(_integer(o1)-_integer(o2));

This code is trying to compare two integers, but the operation will cause the result to be an overflow -ve number and the result of the comparison will be false.

On line 201 the comparison of floats is correct:
_RET_SUCCEED((_float(o1)<_float(o2))?-1:1);

Line 199 should be modified to read:
_RET_SUCCEED((_integer(o1)<_integer(o2))?-1:(_integer(o1)==_integer(o2))?0:1);

I am pretty sure this is a bug this time.

[Kogut]

It may be kind of integer overflow.
http://en.wikipedia.org/wiki/Integer_overflow

[Attila7]

Umm, not sure why you posted the link. I know it is an integer overflow problem as stated:

This code is trying to compare two integers, but the operation will cause the result to be an overflow -ve number and the result of the comparison will be false.

I have this fixed in my version and it now works fine.

Note that although I am using the max -ve 32-bit number in my example, the comparison will give the wrong result whenever you compare A to B where A - B > 2147483647.