cpp/openttd-patchpack/source Changeset - r5399:600219ad4354

Changeset - r5399:600219ad4354

Parent rev.

Child rev.

[Not reviewed]

master

0 2 0

KUDr - 18 years ago 2006-12-28 11:52:12
kudr@openttd.org

(svn r7590) -Codechange: (r7585)
- use existing struct Rect instead of declaring new StationRect
- use inline functions instead of macros

2 files changed with 26 insertions and 35 deletions:

station.h

station_cmd.c

0 comments (0 inline, 0 general)

station.h

➞

Show inline comments

@@ @@ -47,15 +47,6 @@ typedef struct StationSpecList { @@
 	uint8  localidx;   /// Station ID within GRF of station
 } StationSpecList;
 /** Station spread out rectangle (not saved) */
 typedef struct StationRect
+{
 	uint16 left;
 	uint16 top;
 	uint16 right;
 	uint16 bottom;
 } StationRect;
 struct Station {
 	TileIndex xy;
 	RoadStop *bus_stops;
@@ @@ -104,7 +95,7 @@ struct Station { @@
 	byte bus_stop_status_obsolete;
 	byte blocked_months_obsolete;
-	StationRect rect;
+	Rect rect; ///< Station spread out rectangle (not saved) maintained by StationRect_xxx() functions
 };
 enum {

station_cmd.c

➞

Show inline comments

@@ @@ -37,9 +37,9 @@ @@
 static void StationRect_Init(Station *st);
 static bool StationRect_IsEmpty(Station *st);
 static bool StationRect_BeforeAddTile(Station *st, TileIndex tile, bool exec);
-static bool StationRect_BeforeAddRect(Station *st, TileIndex tile, uint16 w, uint16 h, bool exec);
+static bool StationRect_BeforeAddRect(Station *st, TileIndex tile, int w, int h, bool exec);
 static bool StationRect_AfterRemoveTile(Station *st, TileIndex tile);
-static bool StationRect_AfterRemoveRect(Station *st, TileIndex tile, uint16 w, uint16 h);
+static bool StationRect_AfterRemoveRect(Station *st, TileIndex tile, int w, int h);
 /**
@@ @@ -744,7 +744,7 @@ static void UpdateStationAcceptance(Stat @@
 static void UpdateStationSignCoord(Station *st)
+{
-	StationRect *r = &st->rect;
+	Rect *r = &st->rect;
 	if (StationRect_IsEmpty(st)) return; // no tiles belong to this station
@@ @@ -3149,14 +3149,14 @@ const ChunkHandler _station_chunk_handle @@
 };
 #define my_min(a, b) (((a) < (b)) ? (a) : (b))
 #define my_max(a, b) (((a) > (b)) ? (a) : (b))
 #define my_value_in_range(v, a, b) ((a) <= (v) && (v) <= (b))
 #define my_pt_in_rect(r, x, y) (my_value_in_range(x, (r)->left, (r)->right) && my_value_in_range(y, (r)->top, (r)->bottom))
 static inline bool PtInRectXY(Rect *r, int x, int y)
+{
 	return (r->left <= x && x <= r->right && r->top <= y && y <= r->bottom);
+}
 static void StationRect_Init(Station *st)
+{
-	StationRect *r = &st->rect;
+	Rect *r = &st->rect;
 	r->left = r->top = r->right = r->bottom = 0;
+}
@@ @@ -3167,20 +3167,20 @@ static bool StationRect_IsEmpty(Station @@
 static bool StationRect_BeforeAddTile(Station *st, TileIndex tile, bool exec)
+{
 	StationRect *r = &st->rect;
 	uint16 x = TileX(tile);
 	uint16 y = TileY(tile);
 	Rect *r = &st->rect;
 	int x = TileX(tile);
 	int y = TileY(tile);
 	if (StationRect_IsEmpty(st)) {
 		// we are adding the first station tile
 		r->left = r->right = x;
 		r->top = r->bottom = y;
-	} else if (!my_pt_in_rect(r, x, y)) {
+	} else if (!PtInRectXY(r, x, y)) {
 		// current rect is not empty and new point is outside this rect
 		// make new spread-out rectangle
-		StationRect new_rect = {my_min(x, r->left), my_min(y, r->top), my_max(x, r->right), my_max(y, r->bottom)};
+		Rect new_rect = {min(x, r->left), min(y, r->top), max(x, r->right), max(y, r->bottom)};
 		// check new rect dimensions against preset max
 		uint16 w = new_rect.right - new_rect.left + 1;
 		uint16 h = new_rect.bottom - new_rect.top + 1;
 		int w = new_rect.right - new_rect.left + 1;
 		int h = new_rect.bottom - new_rect.top + 1;
 		if (w > _patches.station_spread || h > _patches.station_spread) {
 			assert(!exec);
 			_error_message = STR_306C_STATION_TOO_SPREAD_OUT;
@@ @@ -3197,16 +3197,16 @@ static bool StationRect_BeforeAddTile(St @@
 	return true;
+}
-static bool StationRect_BeforeAddRect(Station *st, TileIndex tile, uint16 w, uint16 h, bool exec)
+static bool StationRect_BeforeAddRect(Station *st, TileIndex tile, int w, int h, bool exec)
+{
 	return StationRect_BeforeAddTile(st, tile, exec) && StationRect_BeforeAddTile(st, TILE_ADDXY(tile, w - 1, h - 1), exec);
+}
-static inline bool ScanRectForStationTiles(StationID st_id, uint16 left, uint16 top, uint16 right, uint16 bottom)
+static inline bool ScanRectForStationTiles(StationID st_id, int left, int top, int right, int bottom)
+{
 	TileIndex top_left = TileXY(left, top);
 	uint16 width = right - left + 1;
 	uint16 height = bottom - top + 1;
 	int width = right - left + 1;
 	int height = bottom - top + 1;
 	BEGIN_TILE_LOOP(tile, width, height, top_left)
 		if (IsTileType(tile, MP_STATION) && GetStationIndex(tile) == st_id) return true;
 	END_TILE_LOOP(tile, width, height, top_left);
@@ @@ -3215,9 +3215,9 @@ static inline bool ScanRectForStationTil @@
 static bool StationRect_AfterRemoveTile(Station *st, TileIndex tile)
+{
 	StationRect *r = &st->rect;
 	uint16 x = TileX(tile);
 	uint16 y = TileY(tile);
 	Rect *r = &st->rect;
 	int x = TileX(tile);
 	int y = TileY(tile);
 	bool reduce_x, reduce_y;
 	// look if removed tile was on the bounding rect edge
@@ @@ -3262,11 +3262,11 @@ static bool StationRect_AfterRemoveTile( @@
 	return false; // non-empty remaining rect
+}
-static bool StationRect_AfterRemoveRect(Station *st, TileIndex tile, uint16 w, uint16 h)
+static bool StationRect_AfterRemoveRect(Station *st, TileIndex tile, int w, int h)
+{
 	bool empty;
 	assert(my_pt_in_rect(&st->rect, TileX(tile), TileY(tile)));
 	assert(my_pt_in_rect(&st->rect, TileX(tile) + w - 1, TileY(tile) + h - 1));
 	assert(PtInRectXY(&st->rect, TileX(tile), TileY(tile)));
 	assert(PtInRectXY(&st->rect, TileX(tile) + w - 1, TileY(tile) + h - 1));
 	empty = StationRect_AfterRemoveTile(st, tile);
 	if (w != 1 || h != 1) empty = empty || StationRect_AfterRemoveTile(st, TILE_ADDXY(tile, w - 1, h - 1));
 	return empty;

0 comments (0 inline, 0 general)