(svn r2758) Add the AB() macro to add a value to a bit range and use it in a few places, also make use of GB and SB nearby
authortron
Sat, 30 Jul 2005 09:29:20 +0000
changeset 2238 14ec09c1a272
parent 2237 5fa9aa2cc5e8
child 2239 5e1245aad0de
(svn r2758) Add the AB() macro to add a value to a bit range and use it in a few places, also make use of GB and SB nearby
macros.h
town_cmd.c
tree_cmd.c
--- a/macros.h	Sat Jul 30 08:15:15 2005 +0000
+++ b/macros.h	Sat Jul 30 09:29:20 2005 +0000
@@ -146,8 +146,10 @@
 
 /// Fetch n bits starting at bit s from x
 #define GB(x, s, n) (((x) >> (s)) & ((1 << (n)) - 1))
-/// Set n bits in x starting at bit s to d
+/// Set n bits starting at bit s in x to d
 #define SB(x, s, n, d) ((x) = ((x) & ~(((1 << (n)) - 1) << (s))) | ((d) << (s)))
+/// Add i to the n bits starting at bit s in x
+#define AB(x, s, n, i) ((x) = ((x) & ~(((1 << (n)) - 1) << (s))) | (((x) + ((i) << (s))) & (((1 << (n)) - 1) << (s))))
 
 /**
  * ROtate x Left/Right by n (must be >= 0)
--- a/town_cmd.c	Sat Jul 30 08:15:15 2005 +0000
+++ b/town_cmd.c	Sat Jul 30 09:29:20 2005 +0000
@@ -180,7 +180,7 @@
 			i = (Random()&7) - 1;
 		} while (i < 0 || i == 1 || i*6==old);
 
-		_m[tile].m5 = (_m[tile].m5 & ~0x3F) | i;
+		SB(_m[tile].m5, 0, 6, i);
 	}
 
 	a = _m[tile].owner&0x7F;
@@ -255,18 +255,12 @@
 
 static void MakeSingleHouseBigger(TileIndex tile)
 {
-	byte b;
-
 	assert(IsTileType(tile, MP_HOUSE));
 
-	b = _m[tile].m5;
-	if (b & 0x80)
-		return;
+	if (_m[tile].m5 & 0x80) return;
 
-	_m[tile].m5 = (b & 0xC0) | ((b+1)&7);
-
-	if ((_m[tile].m5&7) != 0)
-		return;
+	AB(_m[tile].m5, 0, 3, 1);
+	if (GB(_m[tile].m5, 0, 3) != 0) return;
 
 	_m[tile].m3 = _m[tile].m3 + 0x40;
 
--- a/tree_cmd.c	Sat Jul 30 08:15:15 2005 +0000
+++ b/tree_cmd.c	Sat Jul 30 09:29:20 2005 +0000
@@ -485,11 +485,8 @@
 	TileLoopClearHelper(tile);
 
 	/* increase counter */
-	{
-		uint16 m2 = _m[tile].m2;
-		_m[tile].m2 = m2 = (m2 & 0xF0) | ((m2 + 1) & 0xF);
-		if ((m2 & 0xF) != 0) return;
-	}
+	AB(_m[tile].m2, 0, 4, 1);
+	if (GB(_m[tile].m2, 0, 4) != 0) return;
 
 	m5 = _m[tile].m5;
 	if (GB(m5, 0, 3) == 3) {