(svn r3213) - Codechange: Clean up handling of road stops, avoiding unnecessary use of pointers and using the *BIT() macros.
authorpeter1138
Thu, 17 Nov 2005 10:12:21 +0000
changeset 2671 457c6cdbf9e7
parent 2670 34970983ef08
child 2672 ea469d7b34ff
(svn r3213) - Codechange: Clean up handling of road stops, avoiding unnecessary use of pointers and using the *BIT() macros.
roadveh_cmd.c
station_cmd.c
--- a/roadveh_cmd.c	Wed Nov 16 22:20:15 2005 +0000
+++ b/roadveh_cmd.c	Thu Nov 17 10:12:21 2005 +0000
@@ -463,21 +463,13 @@
 
 static void ClearCrashedStation(Vehicle *v)
 {
-	TileIndex tile = v->tile;
-	byte *b, bb;
-
-	RoadStop *rs = GetRoadStopByTile(tile, GetRoadStopType(tile));
-	b = &rs->status;
-
-	bb = *b;
+	RoadStop *rs = GetRoadStopByTile(v->tile, GetRoadStopType(v->tile));
 
 	// mark station as not busy
-	bb &= ~0x80;
+	CLRBIT(rs->status, 7);
 
 	// free parking bay
-	bb |= (v->u.road.state&0x02)?2:1;
-
-	*b = bb;
+	SETBIT(rs->status, HASBIT(v->u.road.state, 1) ? 1 : 0);
 }
 
 static void RoadVehDelete(Vehicle *v)
--- a/station_cmd.c	Wed Nov 16 22:20:15 2005 +0000
+++ b/station_cmd.c	Thu Nov 17 10:12:21 2005 +0000
@@ -2284,34 +2284,30 @@
 			}
 		}
 	} else if (v->type == VEH_Road) {
-		if (v->u.road.state < 16 && (v->u.road.state&4)==0 && v->u.road.frame==0) {
-			byte m5 = _m[tile].m5;
-			byte *b, bb,state;
-
-			if (IS_BYTE_INSIDE(m5, 0x43, 0x4B)) {
+		if (v->u.road.state < 16 && !HASBIT(v->u.road.state, 2) && v->u.road.frame == 0) {
+			if (IS_BYTE_INSIDE(_m[tile].m5, 0x43, 0x4B)) {
+				/* Attempt to allocate a parking bay in a road stop */
 				RoadStop *rs = GetRoadStopByTile(tile, GetRoadStopType(tile));
-				b = &rs->status;
-
-				bb = *b;
-
-				/* bb bits 1..0 describe current the two parking spots.
-				0 means occupied, 1 means free. */
-
-				// Station busy?
-				if (bb & 0x80 || (bb&3) == 0)
+
+				/* rs->status bits 0 and 1 describe current the two parking spots.
+				 * 0 means occupied, 1 means free. */
+
+				// Check if station is busy or if there are no free bays.
+				if (HASBIT(rs->status, 7) || GB(rs->status, 0, 2) == 0)
 					return 8;
 
-				state = v->u.road.state + 32;
-				if (bb & 1) {
-					bb &= ~1;
+				v->u.road.state += 32;
+
+				// if the first bay is free, allocate that, else the second bay must be free.
+				if (HASBIT(rs->status, 0)) {
+					CLRBIT(rs->status, 0);
 				} else {
-					bb &= ~2;
-					state += 2;
+					CLRBIT(rs->status, 1);
+					v->u.road.state += 2;
 				}
 
-				bb |= 0x80;
-				*b = bb;
-				v->u.road.state = state;
+				// mark the station as busy
+				SETBIT(rs->status, 7);
 			}
 		}
 	}