From dbca9eedf66d3fd4f5650de4c3c64c159964a56a Mon Sep 17 00:00:00 2001 From: Steve Checkoway Date: Mon, 5 Jan 2009 06:17:44 +0000 Subject: [PATCH] Revert. IF is a macro, if is a conditional statement. Furthermore, casting to an unsigned 32 bit integer on a 64 bit machine causes the the x<0 to always be false. But since there is a comma, the "bad integer" was evaluating to a nonzero address and so the conditional was always true and the empty statement would be executed. That is wrong. --- stepmania/src/lua-5.1/src/lundump.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/stepmania/src/lua-5.1/src/lundump.c b/stepmania/src/lua-5.1/src/lundump.c index 5d9f2b7179..f1fe7e54c8 100644 --- a/stepmania/src/lua-5.1/src/lundump.c +++ b/stepmania/src/lua-5.1/src/lundump.c @@ -67,7 +67,7 @@ static int LoadInt(LoadState* S) LoadVar(S,x); if (S->flip) x = Swap32(x); - if ((uint32_t)x<0, "bad integer"); + IF ((int32_t)x<0, "bad integer"); return x; }