From 1a9d5ae076fb507cc6056f1da95e1789d2e0b1d9 Mon Sep 17 00:00:00 2001 From: frosch Date: Mon, 2 Aug 2010 23:35:47 +0000 Subject: (svn r20333) -Fix (r20332): Mask second operand to 5 bits to avoid differences between platforms. --- src/newgrf_spritegroup.cpp | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'src/newgrf_spritegroup.cpp') diff --git a/src/newgrf_spritegroup.cpp b/src/newgrf_spritegroup.cpp index 3a543cf0c..6eb136598 100644 --- a/src/newgrf_spritegroup.cpp +++ b/src/newgrf_spritegroup.cpp @@ -124,9 +124,9 @@ static U EvalAdjustT(const DeterministicSpriteGroupAdjust *adjust, ResolverObjec case DSGA_OP_ROR: return RotateRight(last_value, value); case DSGA_OP_SCMP: return ((S)last_value == (S)value) ? 1 : ((S)last_value < (S)value ? 0 : 2); case DSGA_OP_UCMP: return ((U)last_value == (U)value) ? 1 : ((U)last_value < (U)value ? 0 : 2); - case DSGA_OP_SHL: return (U)last_value << (U)value; - case DSGA_OP_SHR: return (U)last_value >> (U)value; - case DSGA_OP_SAR: return (S)last_value >> (U)value; + case DSGA_OP_SHL: return (U)last_value << ((U)value & 0x1F); // mask 'value' to 5 bits, which should behave the same on all architectures. + case DSGA_OP_SHR: return (U)last_value >> ((U)value & 0x1F); + case DSGA_OP_SAR: return (S)last_value >> ((U)value & 0x1F); default: return value; } } -- cgit v1.2.3-54-g00ecf