cpp/openttd-patchpack/source Changeset - r15658:82566ed4deb1

Changeset - r15658:82566ed4deb1

Parent rev.

Child rev.

[Not reviewed]

master

0 1 0

frosch - 14 years ago 2010-08-02 23:35:47
frosch@openttd.org

(svn r20333) -Fix (r20332): Mask second operand to 5 bits to avoid differences between platforms.

1 file changed with 3 insertions and 3 deletions:

src/newgrf_spritegroup.cpp

0 comments (0 inline, 0 general)

src/newgrf_spritegroup.cpp

➞

Show inline comments

@@ @@ -124,9 +124,9 @@ static U EvalAdjustT(const Deterministic @@
 		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;
+	}
+}

0 comments (0 inline, 0 general)