summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorKUDr <kudr@openttd.org>2006-12-28 11:52:12 +0000
committerKUDr <kudr@openttd.org>2006-12-28 11:52:12 +0000
commit02c07d2a86f6f585324f775a72f0706afe8e3569 (patch)
tree574a77b81258595661dcfcdfc6e76d69ad495301
parentd657aa97de44adc8a1957529718eebedbebc3ba9 (diff)
downloadopenttd-02c07d2a86f6f585324f775a72f0706afe8e3569.tar.xz
(svn r7590) -Codechange: (r7585)
- use existing struct Rect instead of declaring new StationRect - use inline functions instead of macros
-rw-r--r--station.h11
-rw-r--r--station_cmd.c50
2 files changed, 26 insertions, 35 deletions
diff --git a/station.h b/station.h
index 1d4c850a8..aaa168384 100644
--- a/station.h
+++ b/station.h
@@ -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 {
diff --git a/station_cmd.c b/station_cmd.c
index cc2eb15e6..091327c25 100644
--- a/station_cmd.c
+++ b/station_cmd.c
@@ -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(Station *st, bool show_msg)
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_handlers[] = {
};
-#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 *st)
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(Station *st, TileIndex tile, bool exec)
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 ScanRectForStationTiles(StationID st_id, uint16 left, uint16
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(Station *st, TileIndex tile)
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;