From a63a84ef5dd560226e50fefdbb8cc33887e85093 Mon Sep 17 00:00:00 2001 From: matthijs Date: Fri, 17 Jun 2005 00:22:46 +0000 Subject: (svn r2450) * Codechange: Replaced all uses of the arrays in tile.h with calls to the associated wrapper functions. * Codechange: Made npf.c use some map array accessing wrappers instead of direct access. * Codechange/Fix: Named every enum in tile.h. Fixes a nasty bug on MSVC where arrays would be initialised with zeroes (tnx Asterix_) * Removed magic numbers from tables in tile.c. * Added some explicit casts in tile.h. --- depot.h | 2 +- npf.c | 59 ++++++++++++++++---------------- rail.c | 76 ++++++++++++++++++++++-------------------- rail.h | 113 ++++++++++++++++++++++++++++++++++++-------------------------- tile.h | 2 +- vehicle.c | 18 +++++----- vehicle.h | 3 +- 7 files changed, 146 insertions(+), 127 deletions(-) diff --git a/depot.h b/depot.h index 502f4f136..da621e732 100644 --- a/depot.h +++ b/depot.h @@ -98,7 +98,7 @@ static inline DiagDirection GetDepotDirection(TileIndex tile, TransportType type case TRANSPORT_RAIL: case TRANSPORT_ROAD: /* Rail and road store a diagonal direction in bits 0 and 1 */ - return _map5[tile] & 3; + return (DiagDirection)(_map5[tile] & 3); case TRANSPORT_WATER: /* Water is stubborn, it stores the directions in a different order. */ switch (_map5[tile] & 3) { diff --git a/npf.c b/npf.c index 3c9cb08c3..ea48a8234 100644 --- a/npf.c +++ b/npf.c @@ -128,7 +128,7 @@ void NPFFillTrackdirChoice(AyStarNode* current, OpenListNode* parent) * cost of that tile. If the tile is an exit, it will return the tunnel length * including the exit tile. Requires that this is a Tunnel tile */ uint NPFTunnelCost(AyStarNode* current) { - byte exitdir = _trackdir_to_exitdir[current->direction]; + byte exitdir = TrackdirToExitdir(current->direction); TileIndex tile = current->tile; if ( (uint)(_map5[tile] & 3) == ReverseDiagdir(exitdir)) { /* We just popped out if this tunnel, since were @@ -145,7 +145,7 @@ uint NPFTunnelCost(AyStarNode* current) { } uint NPFSlopeCost(AyStarNode* current) { - TileIndex next = current->tile + TileOffsByDir(_trackdir_to_exitdir[current->direction]); + TileIndex next = current->tile + TileOffsByDir(TrackdirToExitdir(current->direction)); int x,y; int8 z1,z2; @@ -206,7 +206,7 @@ int32 NPFWaterPathCost(AyStar* as, AyStarNode* current, OpenListNode* parent) { if (IsBuoyTile(current->tile) && IsDiagonalTrackdir(current->direction)) cost += _patches.npf_buoy_penalty; /* A small penalty for going over buoys */ - if (current->direction != _next_trackdir[parent->path.node.direction]) + if (current->direction != NextTrackdir(parent->path.node.direction)) cost += _patches.npf_water_curve_penalty; /* TODO More penalties? */ @@ -291,16 +291,17 @@ int32 NPFRailPathCost(AyStar* as, AyStarNode* current, OpenListNode* parent) { /* Determine extra costs */ /* Check for signals */ - if (IsTileType(tile, MP_RAILWAY) && (_map5[tile] & 0xC0) == 0x40 && (_map3_lo[tile] & _signal_along_trackdir[trackdir]) != 0) { + if (IsTileType(tile, MP_RAILWAY) && HasSignalOnTrackdir(tile, trackdir)) { /* Ordinary track with signals */ - if ((_map2[tile] & _signal_along_trackdir[trackdir]) == 0) { + if (GetSignalState(tile, trackdir) == SIGNAL_STATE_RED) { /* Signal facing us is red */ if (!NPFGetFlag(current, NPF_FLAG_SEEN_SIGNAL)) { /* Penalize the first signal we * encounter, if it is red */ /* Is this a presignal exit or combo? */ - if ((_map3_hi[tile] & 0x3) == 0x2 || (_map3_hi[tile] & 0x3) == 0x3) + SignalType sigtype = GetSignalType(tile, trackdir); + if (sigtype == SIGTYPE_EXIT || sigtype == SIGTYPE_COMBO) /* Penalise exit and combo signals differently (heavier) */ cost += _patches.npf_rail_firstred_exit_penalty; else @@ -325,7 +326,7 @@ int32 NPFRailPathCost(AyStar* as, AyStarNode* current, OpenListNode* parent) { cost += NPFSlopeCost(current); /* Check for turns */ - if (current->direction != _next_trackdir[parent->path.node.direction]) + if (current->direction != NextTrackdir(parent->path.node.direction)) cost += _patches.npf_rail_curve_penalty; //TODO, with realistic acceleration, also the amount of straight track between // curves should be taken into account, as this affects the speed limit. @@ -436,20 +437,20 @@ static inline RailType GetTileRailType(TileIndex tile, byte trackdir) * AyStarNode.user_data[NPF_TRACKDIR_CHOICE] with an appropriate value, and * copy AyStarNode.user_data[NPF_NODE_FLAGS] from the parent */ void NPFFollowTrack(AyStar* aystar, OpenListNode* current) { - byte src_trackdir = current->path.node.direction; + Trackdir src_trackdir = current->path.node.direction; TileIndex src_tile = current->path.node.tile; - byte src_exitdir = _trackdir_to_exitdir[src_trackdir]; + DiagDirection src_exitdir = TrackdirToExitdir(src_trackdir); FindLengthOfTunnelResult flotr; TileIndex dst_tile; int i = 0; - uint trackdirs, ts; + TrackdirBits trackdirbits, ts; TransportType type = aystar->user_data[NPF_TYPE]; /* Initialize to 0, so we can jump out (return) somewhere an have no neighbours */ aystar->num_neighbours = 0; DEBUG(npf, 4)("Expanding: (%d, %d, %d) [%d]", TileX(src_tile), TileY(src_tile), src_trackdir, src_tile); /* Find dest tile */ - if (IsTileType(src_tile, MP_TUNNELBRIDGE) && (_map5[src_tile] & 0xF0)==0 && (_map5[src_tile] & 3) == src_exitdir) { + if (IsTileType(src_tile, MP_TUNNELBRIDGE) && (_map5[src_tile] & 0xF0)==0 && (DiagDirection)(_map5[src_tile] & 3) == src_exitdir) { /* This is a tunnel. We know this tunnel is our type, * otherwise we wouldn't have got here. It is also facing us, * so we should skip it's body */ @@ -472,7 +473,7 @@ void NPFFollowTrack(AyStar* aystar, OpenListNode* current) { * otherwise (only for trains, since only with trains you can * (sometimes) reach tiles after reversing that you couldn't reach * without reversing. */ - if (src_trackdir == _dir_to_diag_trackdir[ReverseDiagdir(exitdir)] && type == TRANSPORT_RAIL) + if (src_trackdir == DiagdirToDiagTrackdir(ReverseDiagdir(exitdir)) && type == TRANSPORT_RAIL) /* We are headed inwards. We can only reverse here, so we'll not * consider this direction, but jump ahead to the reverse direction. * It would be nicer to return one neighbour here (the reverse @@ -480,10 +481,10 @@ void NPFFollowTrack(AyStar* aystar, OpenListNode* current) { * that one to return the tracks outside of the depot. But, because * the code layout is cleaner this way, we will just pretend we are * reversed already */ - src_trackdir = _reverse_trackdir[src_trackdir]; + src_trackdir = ReverseTrackdir(src_trackdir); } /* This a normal tile, a bridge, a tunnel exit, etc. */ - dst_tile = AddTileIndexDiffCWrap(src_tile, TileIndexDiffCByDir(_trackdir_to_exitdir[src_trackdir])); + dst_tile = AddTileIndexDiffCWrap(src_tile, TileIndexDiffCByDir(TrackdirToExitdir(src_trackdir))); if (dst_tile == INVALID_TILE) { /* We reached the border of the map */ /* TODO Nicer control flow for this */ @@ -497,7 +498,7 @@ void NPFFollowTrack(AyStar* aystar, OpenListNode* current) { * the type of the vehicle... Maybe an NPF_RAILTYPE userdata sometime? */ if (type == TRANSPORT_RAIL) { byte src_type = GetTileRailType(src_tile, src_trackdir); - byte dst_type = GetTileRailType(dst_tile, _trackdir_to_exitdir[src_trackdir]); + byte dst_type = GetTileRailType(dst_tile, TrackdirToExitdir(src_trackdir)); if (src_type != dst_type) { return; } @@ -531,31 +532,27 @@ void NPFFollowTrack(AyStar* aystar, OpenListNode* current) { } else { ts = GetTileTrackStatus(dst_tile, type); } - trackdirs = ts & 0x3F3F; /* Filter out signal status and the unused bits */ + trackdirbits = ts & 0x3F3F; /* Filter out signal status and the unused bits */ - DEBUG(npf, 4)("Next node: (%d, %d) [%d], possible trackdirs: %#x", TileX(dst_tile), TileY(dst_tile), dst_tile, trackdirs); + DEBUG(npf, 4)("Next node: (%d, %d) [%d], possible trackdirs: %#x", TileX(dst_tile), TileY(dst_tile), dst_tile, trackdirbits); /* Select only trackdirs we can reach from our current trackdir */ - trackdirs &= TrackdirReachesTrackdirs(src_trackdir); + trackdirbits &= TrackdirReachesTrackdirs(src_trackdir); if (_patches.forbid_90_deg && (type == TRANSPORT_RAIL || type == TRANSPORT_WATER)) /* Filter out trackdirs that would make 90 deg turns for trains */ - trackdirs &= ~_trackdir_crosses_trackdirs[src_trackdir]; - DEBUG(npf,6)("After filtering: (%d, %d), possible trackdirs: %#x", TileX(dst_tile), TileY(dst_tile), trackdirs); + trackdirbits &= ~TrackdirCrossesTrackdirs(src_trackdir); + DEBUG(npf,6)("After filtering: (%d, %d), possible trackdirs: %#x", TileX(dst_tile), TileY(dst_tile), trackdirbits); /* Enumerate possible track */ - while (trackdirs != 0) { + while (trackdirbits != 0) { byte dst_trackdir; - dst_trackdir = FindFirstBit2x64(trackdirs); - trackdirs = KillFirstBit2x64(trackdirs); - DEBUG(npf, 5)("Expanded into trackdir: %d, remaining trackdirs: %#x", dst_trackdir, trackdirs); + dst_trackdir = FindFirstBit2x64(trackdirbits); + trackdirbits = KillFirstBit2x64(trackdirbits); + DEBUG(npf, 5)("Expanded into trackdir: %d, remaining trackdirs: %#x", dst_trackdir, trackdirbits); /* Check for oneway signal against us */ - if (IsTileType(dst_tile, MP_RAILWAY) && (_map5[dst_tile]&0xC0) == 0x40) { - // the tile has a signal - byte signal_present = _map3_lo[dst_tile]; - if (!(signal_present & _signal_along_trackdir[dst_trackdir])) { + if (IsTileType(dst_tile, MP_RAILWAY) && GetRailTileType(dst_tile) == RAIL_TYPE_SIGNALS) { + if (HasSignalOnTrackdir(dst_tile, ReverseTrackdir(dst_trackdir)) && !HasSignalOnTrackdir(dst_tile, dst_trackdir)) // if one way signal not pointing towards us, stop going in this direction. - if (signal_present & _signal_against_trackdir[dst_trackdir]) - break; - } + break; } { /* We've found ourselves a neighbour :-) */ diff --git a/rail.c b/rail.c index 24f383b50..58dd0610d 100644 --- a/rail.c +++ b/rail.c @@ -1,5 +1,6 @@ #include "rail.h" +/* XXX: Below 3 tables store duplicate data. Maybe remove some? */ /* Maps a trackdir to the bit that stores its status in the map arrays, in the * direction along with the trackdir */ const byte _signal_along_trackdir[] = { @@ -24,70 +25,71 @@ const byte _signal_on_track[] = { * track entering in this direction (including those making 90 degree turns) */ const TrackdirBits _exitdir_reaches_trackdirs[] = { - TRACKDIR_BIT_DIAG1_NE|TRACKDIR_BIT_LOWER_E|TRACKDIR_BIT_LEFT_N, /* DIAGDIR_NE */ - TRACKDIR_BIT_DIAG2_SE|TRACKDIR_BIT_LEFT_S |TRACKDIR_BIT_UPPER_E, /* DIAGDIR_SE */ - TRACKDIR_BIT_DIAG1_SW|TRACKDIR_BIT_UPPER_W|TRACKDIR_BIT_RIGHT_S, /* DIAGDIR_SW */ - TRACKDIR_BIT_DIAG2_NW|TRACKDIR_BIT_RIGHT_N|TRACKDIR_BIT_LOWER_W /* DIAGDIR_NW */ + TRACKDIR_BIT_DIAG1_NE | TRACKDIR_BIT_LOWER_E | TRACKDIR_BIT_LEFT_N, /* DIAGDIR_NE */ + TRACKDIR_BIT_DIAG2_SE | TRACKDIR_BIT_LEFT_S | TRACKDIR_BIT_UPPER_E, /* DIAGDIR_SE */ + TRACKDIR_BIT_DIAG1_SW | TRACKDIR_BIT_UPPER_W | TRACKDIR_BIT_RIGHT_S, /* DIAGDIR_SW */ + TRACKDIR_BIT_DIAG2_NW | TRACKDIR_BIT_RIGHT_N | TRACKDIR_BIT_LOWER_W /* DIAGDIR_NW */ }; -/* TODO: Remove magic numbers from tables below just like - * _exitdir_reaches_trackdirs[] */ - -const Trackdir _next_trackdir[14] = { - 0, 1, 3, 2, 5, 4, 0, 0, - 8, 9, 11, 10, 13, 12 +const Trackdir _next_trackdir[] = { + TRACKDIR_DIAG1_NE, TRACKDIR_DIAG2_SE, TRACKDIR_LOWER_E, TRACKDIR_UPPER_E, TRACKDIR_RIGHT_S, TRACKDIR_LEFT_S, INVALID_TRACKDIR, INVALID_TRACKDIR, + TRACKDIR_DIAG1_SW, TRACKDIR_DIAG2_NW, TRACKDIR_LOWER_W, TRACKDIR_UPPER_W, TRACKDIR_RIGHT_N, TRACKDIR_LEFT_N }; /* Maps a trackdir to all trackdirs that make 90 deg turns with it. */ -const TrackdirBits _trackdir_crosses_trackdirs[] = { - 0x0202, 0x0101, 0x3030, 0x3030, 0x0C0C, 0x0C0C, 0, 0, - 0x0202, 0x0101, 0x3030, 0x3030, 0x0C0C, 0x0C0C +const TrackdirBits _track_crosses_trackdirs[] = { + TRACKDIR_BIT_DIAG2_SE | TRACKDIR_BIT_DIAG2_NW, /* TRACK_DIAG1 */ + TRACKDIR_BIT_DIAG1_NE | TRACKDIR_BIT_DIAG1_SW, /* TRACK_DIAG2 */ + TRACKDIR_BIT_RIGHT_N | TRACKDIR_BIT_RIGHT_S | TRACKDIR_BIT_LEFT_N | TRACKDIR_BIT_LEFT_S, /* TRACK_UPPER */ + TRACKDIR_BIT_RIGHT_N | TRACKDIR_BIT_RIGHT_S | TRACKDIR_BIT_LEFT_N | TRACKDIR_BIT_LEFT_S, /* TRACK_LOWER */ + TRACKDIR_BIT_UPPER_W | TRACKDIR_BIT_UPPER_E | TRACKDIR_BIT_LOWER_W | TRACKDIR_BIT_LOWER_E, /* TRACK_LEFT */ + TRACKDIR_BIT_UPPER_W | TRACKDIR_BIT_UPPER_E | TRACKDIR_BIT_LOWER_W | TRACKDIR_BIT_LOWER_E /* TRACK_RIGHT */ }; /* Maps a track to all tracks that make 90 deg turns with it. */ const TrackBits _track_crosses_tracks[] = { - 0x2, /* Track 1 -> Track 2 */ - 0x1, /* Track 2 -> Track 1 */ - 0x30, /* Upper -> Left | Right */ - 0x30, /* Lower -> Left | Right */ - 0x0C, /* Left -> Upper | Lower */ - 0x0C, /* Right -> Upper | Lower */ + TRACK_BIT_DIAG2, /* TRACK_DIAG1 */ + TRACK_BIT_DIAG1, /* TRACK_DIAG2 */ + TRACK_BIT_LEFT | TRACK_BIT_RIGHT, /* TRACK_UPPER */ + TRACK_BIT_LEFT | TRACK_BIT_RIGHT, /* TRACK_LOWER */ + TRACK_BIT_UPPER | TRACK_BIT_LOWER, /* TRACK_LEFT */ + TRACK_BIT_UPPER | TRACK_BIT_LOWER /* TRACK_RIGHT */ }; /* Maps a trackdir to the (4-way) direction the tile is exited when following * that trackdir */ const DiagDirection _trackdir_to_exitdir[] = { - 0,1,0,1,2,1, 0,0, - 2,3,3,2,3,0, + DIAGDIR_NE,DIAGDIR_SE,DIAGDIR_NE,DIAGDIR_SE,DIAGDIR_SW,DIAGDIR_SE, DIAGDIR_NE,DIAGDIR_NE, + DIAGDIR_SW,DIAGDIR_NW,DIAGDIR_NW,DIAGDIR_SW,DIAGDIR_NW,DIAGDIR_NE, }; const Trackdir _track_exitdir_to_trackdir[][DIAGDIR_END] = { - {0, 0xff, 8, 0xff}, - {0xff, 1, 0xff, 9}, - {2, 0xff, 0xff, 10}, - {0xff, 3, 11, 0xf}, - {0xff, 0xff, 4, 12}, - {13, 5, 0xff, 0xff} + {TRACKDIR_DIAG1_NE, INVALID_TRACKDIR, TRACKDIR_DIAG1_SW, INVALID_TRACKDIR}, + {INVALID_TRACKDIR, TRACKDIR_DIAG2_SE, INVALID_TRACKDIR, TRACKDIR_DIAG2_NW}, + {TRACKDIR_UPPER_E, INVALID_TRACKDIR, INVALID_TRACKDIR, TRACKDIR_UPPER_W}, + {INVALID_TRACKDIR, TRACKDIR_LOWER_E, TRACKDIR_LOWER_W, INVALID_TRACKDIR}, + {INVALID_TRACKDIR, INVALID_TRACKDIR, TRACKDIR_LEFT_S, TRACKDIR_LEFT_N}, + {TRACKDIR_RIGHT_N, TRACKDIR_RIGHT_S, INVALID_TRACKDIR, INVALID_TRACKDIR} }; const Trackdir _track_direction_to_trackdir[][DIR_END] = { - {0xff, 0, 0xff, 0xff, 0xff, 8, 0xff, 0xff}, - {0xff, 0xff, 0xff, 1, 0xff, 0xff, 0xff, 9}, - {0xff, 0xff, 2, 0xff, 0xff, 0xff, 10, 0xff}, - {0xff, 0xff, 3, 0xff, 0xff, 0xff, 11, 0xff}, - {12, 0xff, 0xff, 0xff, 4, 0xff, 0xff, 0xff}, - {13, 0xff, 0xff, 0xff, 5, 0xff, 0xff, 0xff} + {INVALID_TRACKDIR, TRACKDIR_DIAG1_NE, INVALID_TRACKDIR, INVALID_TRACKDIR, INVALID_TRACKDIR, TRACKDIR_DIAG1_SW, INVALID_TRACKDIR, INVALID_TRACKDIR}, + {INVALID_TRACKDIR, INVALID_TRACKDIR, INVALID_TRACKDIR, TRACKDIR_DIAG2_SE, INVALID_TRACKDIR, INVALID_TRACKDIR, INVALID_TRACKDIR, TRACKDIR_DIAG2_NW}, + {INVALID_TRACKDIR, INVALID_TRACKDIR, TRACKDIR_UPPER_E, INVALID_TRACKDIR, INVALID_TRACKDIR, INVALID_TRACKDIR, TRACKDIR_UPPER_W, INVALID_TRACKDIR}, + {INVALID_TRACKDIR, INVALID_TRACKDIR, TRACKDIR_LOWER_E, INVALID_TRACKDIR, INVALID_TRACKDIR, INVALID_TRACKDIR, TRACKDIR_LOWER_W, INVALID_TRACKDIR}, + {TRACKDIR_LEFT_N, INVALID_TRACKDIR, INVALID_TRACKDIR, INVALID_TRACKDIR, TRACKDIR_LEFT_S, INVALID_TRACKDIR, INVALID_TRACKDIR, INVALID_TRACKDIR}, + {TRACKDIR_RIGHT_N, INVALID_TRACKDIR, INVALID_TRACKDIR, INVALID_TRACKDIR, TRACKDIR_RIGHT_S, INVALID_TRACKDIR, INVALID_TRACKDIR, INVALID_TRACKDIR} }; const Trackdir _dir_to_diag_trackdir[] = { - 0, 1, 8, 9, + TRACKDIR_DIAG1_NE, TRACKDIR_DIAG2_SE, TRACKDIR_DIAG1_SW, TRACKDIR_DIAG2_NW, }; const DiagDirection _reverse_diagdir[] = { - 2, 3, 0, 1 + DIAGDIR_SW, DIAGDIR_NW, DIAGDIR_NE, DIAGDIR_SE }; const Trackdir _reverse_trackdir[] = { - 8, 9, 10, 11, 12, 13, 0xFF, 0xFF, - 0, 1, 2, 3, 4, 5 + TRACKDIR_DIAG1_SW, TRACKDIR_DIAG2_NW, TRACKDIR_UPPER_W, TRACKDIR_LOWER_W, TRACKDIR_LEFT_N, TRACKDIR_RIGHT_N, INVALID_TRACKDIR, INVALID_TRACKDIR, + TRACKDIR_DIAG1_NE, TRACKDIR_DIAG2_SE, TRACKDIR_UPPER_E, TRACKDIR_LOWER_E, TRACKDIR_LEFT_S, TRACKDIR_RIGHT_S }; diff --git a/rail.h b/rail.h index e2d13baa6..61c81831b 100644 --- a/rail.h +++ b/rail.h @@ -33,13 +33,13 @@ enum { /* DEPRECATED TODO: Rewrite all uses of this */ /* These subtypes are used in the map5 byte when the main rail type is * RAIL_TYPE_DEPOT_WAYPOINT */ -typedef enum { +typedef enum RailTileSubtypes { RAIL_SUBTYPE_DEPOT = 0x00, RAIL_SUBTYPE_WAYPOINT = 0x04, RAIL_SUBTYPE_MASK = 0x3C, } RailTileSubtype; -typedef enum { +typedef enum SignalTypes { /* Stored in _map3_hi[0..1] for MP_RAILWAY */ SIGTYPE_NORMAL = 0, // normal signal SIGTYPE_ENTRY = 1, // presignal block entry @@ -49,7 +49,7 @@ typedef enum { SIGTYPE_MASK = 3, } SignalType; -typedef enum { +typedef enum RailTypes { RAILTYPE_RAIL = 0, RAILTYPE_MONO = 1, RAILTYPE_MAGLEV = 2, @@ -64,7 +64,7 @@ enum { /* These are used to specify a single track. Can be translated to a trackbit * with TrackToTrackbit */ -typedef enum { +typedef enum Tracks { TRACK_DIAG1 = 0, TRACK_DIAG2 = 1, TRACK_UPPER = 2, @@ -76,7 +76,7 @@ typedef enum { } Track; /* These are the bitfield variants of the above */ -typedef enum { +typedef enum TrackBits { TRACK_BIT_DIAG1 = 1, // 0 TRACK_BIT_DIAG2 = 2, // 1 TRACK_BIT_UPPER = 4, // 2 @@ -88,7 +88,7 @@ typedef enum { /* These are a combination of tracks and directions. Values are 0-5 in one direction (corresponding to the Track enum) and 8-13 in the other direction. */ -typedef enum { +typedef enum Trackdirs { TRACKDIR_DIAG1_NE = 0, TRACKDIR_DIAG2_SE = 1, TRACKDIR_UPPER_E = 2, @@ -108,7 +108,7 @@ typedef enum { /* These are a combination of tracks and directions. Values are 0-5 in one direction (corresponding to the Track enum) and 8-13 in the other direction. */ -typedef enum { +typedef enum TrackdirBits { TRACKDIR_BIT_DIAG1_NE = 0x1, TRACKDIR_BIT_DIAG2_SE = 0x2, TRACKDIR_BIT_UPPER_E = 0x4, @@ -130,9 +130,9 @@ typedef enum { * simple boolean logic will do. But do try to compare to this enum instead of * normal boolean evaluation, since that will make future additions easier. */ -typedef enum { - SIGNALSTATE_RED = 0, - SIGNALSTATE_GREEN = 1, +typedef enum SignalStates { + SIGNAL_STATE_RED = 0, + SIGNAL_STATE_GREEN = 1, } SignalState; @@ -187,7 +187,7 @@ static inline RailTileType GetRailTileType(TileIndex tile) /** * Returns the rail type of the given rail tile (ie rail, mono, maglev). */ -static inline RailType GetRailType(TileIndex tile) { return _map3_lo[tile] & RAILTYPE_MASK; } +static inline RailType GetRailType(TileIndex tile) { return (RailType)(_map3_lo[tile] & RAILTYPE_MASK); } /** * Checks if a rail tile has signals. @@ -204,7 +204,7 @@ static inline bool HasSignals(TileIndex tile) static inline RailTileSubtype GetRailTileSubtype(TileIndex tile) { assert(GetRailTileType(tile) == RAIL_TYPE_DEPOT_WAYPOINT); - return _map5[tile] & RAIL_SUBTYPE_MASK; + return (RailTileSubtype)(_map5[tile] & RAIL_SUBTYPE_MASK); } /** @@ -223,7 +223,7 @@ static inline bool IsPlainRailTile(TileIndex tile) static inline TrackBits GetTrackBits(TileIndex tile) { assert(GetRailTileType(tile) == RAIL_TYPE_NORMAL || GetRailTileType(tile) == RAIL_TYPE_SIGNALS); - return _map5[tile] & TRACK_BIT_MASK; + return (TrackBits)(_map5[tile] & TRACK_BIT_MASK); } /** @@ -244,6 +244,37 @@ static inline bool HasTrack(TileIndex tile, Track track) * from the global scope and expose direct uses of them. */ +/** + * Maps a trackdir to the reverse trackdir. + */ +const Trackdir _reverse_trackdir[TRACKDIR_END]; +static inline Trackdir ReverseTrackdir(Trackdir trackdir) { return _reverse_trackdir[trackdir]; } + +/** + * Maps a Trackdir to the corresponding TrackdirBits value + */ +static inline TrackdirBits TrackdirToTrackdirBits(Trackdir trackdir) { return (TrackdirBits)(1 << trackdir); } + +/* + * Maps a Track to the corresponding TrackBits value + */ +static inline TrackBits TrackToTrackBits(Track track) { return (TrackBits)(1 << track); } + +/* Returns the Track that a given Trackdir represents */ +static inline Track TrackdirToTrack(Trackdir trackdir) { return (Track)(trackdir & 0x7); } + +/* Returns a Trackdir for the given Track. Since every Track corresponds to + * two Trackdirs, we choose the one which points between NE and S. + * Note that the actual implementation is quite futile, but this might change + * in the future. + */ +static inline Trackdir TrackToTrackdir(Track track) { return (Trackdir)track; } + +/* Returns a TrackdirBit mask that contains the two TrackdirBits that + * correspond with the given Track (one for each direction). + */ +static inline TrackdirBits TrackToTrackdirBits(Track track) { Trackdir td = TrackToTrackdir(track); return TrackdirToTrackdirBits(td) | TrackdirToTrackdirBits(ReverseTrackdir(td));} + /** * Maps a trackdir to the trackdir that you will end up on if you go straight * ahead. This will be the same trackdir for diagonal trackdirs, but a @@ -298,8 +329,8 @@ static inline TrackdirBits TrackdirReachesTrackdirs(Trackdir trackdir) { return /** * Maps a trackdir to all trackdirs that make 90 deg turns with it. */ -const TrackdirBits _trackdir_crosses_trackdirs[TRACKDIR_END]; -static inline TrackdirBits TrackdirCrossesTrackdirs(Trackdir trackdir) { return _trackdir_crosses_trackdirs[trackdir]; } +const TrackdirBits _track_crosses_trackdirs[TRACKDIR_END]; +static inline TrackdirBits TrackdirCrossesTrackdirs(Trackdir trackdir) { return _track_crosses_trackdirs[TrackdirToTrack(trackdir)]; } /** * Maps a (4-way) direction to the reverse. @@ -307,34 +338,8 @@ static inline TrackdirBits TrackdirCrossesTrackdirs(Trackdir trackdir) { return const DiagDirection _reverse_diagdir[DIAGDIR_END]; static inline DiagDirection ReverseDiagdir(DiagDirection diagdir) { return _reverse_diagdir[diagdir]; } -/** - * Maps a trackdir to the reverse trackdir. - */ -const Trackdir _reverse_trackdir[TRACKDIR_END]; -static inline Trackdir ReverseTrackdir(Trackdir trackdir) { return _reverse_trackdir[trackdir]; } - -/** - * Maps a Trackdir to the corresponding TrackdirBits value - */ -static inline TrackdirBits TrackdirToTrackdirBits(Trackdir trackdir) { return 1 << trackdir; } - -/* - * Maps a Track to the corresponding TrackBits value - */ -static inline TrackBits TrackToTrackBits(Track track) { return 1 << track; } - -/* Returns the Track that a given Trackdir represents */ -static inline Track TrackdirToTrack(Trackdir trackdir) { return trackdir & 0x7; } - -/* Returns a Trackdir for the given Track. Since every Track corresponds to - * two Trackdirs, we choose the one which points between N and SE. - * Note that the actual implementation is quite futile, but this might change - * in the future. - */ -static inline Trackdir TrackToTrackdir(Track track) { return track; } - /* Checks if a given Track is diagonal */ -static inline bool IsDiagonalTrack(Track track) { return track == TRACK_DIAG1 || track == TRACK_DIAG2; } +static inline bool IsDiagonalTrack(Track track) { return (track == TRACK_DIAG1) || (track == TRACK_DIAG2); } /* Checks if a given Trackdir is diagonal. */ static inline bool IsDiagonalTrackdir(Trackdir trackdir) { return IsDiagonalTrack(TrackdirToTrack(trackdir)); } @@ -344,12 +349,26 @@ static inline bool IsDiagonalTrackdir(Trackdir trackdir) { return IsDiagonalTrac */ /** - * Checks for the presence of signals on the given track on the given tile + * Checks for the presence of signals (either way) on the given track on the + * given rail tile. */ static inline bool HasSignalOnTrack(TileIndex tile, Track track) { assert(IsValidTrack(track)); - return (GetRailTileType(tile) == RAIL_TYPE_SIGNALS && (_map3_lo[tile] & SignalOnTrack(track))); + return ((GetRailTileType(tile) == RAIL_TYPE_SIGNALS) && ((_map3_lo[tile] & SignalOnTrack(track)) != 0)); +} + +/** + * Checks for the presence of signals along the given trackdir on the given + * rail tile. + * + * Along meaning if you are currently driving on the given trackdir, this is + * the signal that is facing us (for which we stop when it's red). + */ +static inline bool HasSignalOnTrackdir(TileIndex tile, Trackdir trackdir) +{ + assert (IsValidTrackdir(trackdir)); + return (GetRailTileType(tile) == RAIL_TYPE_SIGNALS) && (_map3_lo[tile] & SignalAlongTrackdir(trackdir)); } /** @@ -362,7 +381,7 @@ static inline SignalState GetSignalState(TileIndex tile, Trackdir trackdir) { assert(IsValidTrackdir(trackdir)); assert(HasSignalOnTrack(tile, TrackdirToTrack(trackdir))); - return ((_map2[tile] & SignalAlongTrackdir(trackdir))?SIGNALSTATE_GREEN:SIGNALSTATE_RED); + return ((_map2[tile] & SignalAlongTrackdir(trackdir))?SIGNAL_STATE_GREEN:SIGNAL_STATE_RED); } /** @@ -376,7 +395,7 @@ static inline SignalType GetSignalType(TileIndex tile, Track track) { assert(IsValidTrack(track)); assert(GetRailTileType(tile) == RAIL_TYPE_SIGNALS); - return _map3_hi[tile] & SIGTYPE_MASK; + return (SignalType)(_map3_hi[tile] & SIGTYPE_MASK); } /** @@ -391,7 +410,7 @@ static inline SignalType GetSignalType(TileIndex tile, Track track) static inline bool HasSemaphores(TileIndex tile, Track track) { assert(IsValidTrack(track)); - return _map3_hi[tile] & SIG_SEMAPHORE_MASK; + return (_map3_hi[tile] & SIG_SEMAPHORE_MASK); } #endif // RAIL_H diff --git a/tile.h b/tile.h index 36c07e750..24bd22dc5 100644 --- a/tile.h +++ b/tile.h @@ -4,7 +4,7 @@ #include "macros.h" #include "map.h" -typedef enum TileType { +typedef enum TileTypes { MP_CLEAR, MP_RAILWAY, MP_STREET, diff --git a/vehicle.c b/vehicle.c index 76ead42cd..7208e807b 100644 --- a/vehicle.c +++ b/vehicle.c @@ -1736,7 +1736,7 @@ byte GetDirectionTowards(Vehicle *v, int x, int y) return (dir+((dirdiff&7)<5?1:-1)) & 7; } -byte GetVehicleTrackdir(const Vehicle* v) +Trackdir GetVehicleTrackdir(const Vehicle* v) { if (v->vehstatus & VS_CRASHED) return 0xFF; @@ -1744,28 +1744,28 @@ byte GetVehicleTrackdir(const Vehicle* v) { case VEH_Train: if (v->u.rail.track == 0x80) /* We'll assume the train is facing outwards */ - return _dir_to_diag_trackdir[GetDepotDirection(v->tile, TRANSPORT_RAIL)]; /* Train in depot */ + return DiagdirToDiagTrackdir(GetDepotDirection(v->tile, TRANSPORT_RAIL)); /* Train in depot */ if (v->u.rail.track == 0x40) /* train in tunnel, so just use his direction and assume a diagonal track */ - return _dir_to_diag_trackdir[(v->direction >> 1) & 3]; + return DiagdirToDiagTrackdir((v->direction >> 1) & 3); - return _track_direction_to_trackdir[FIND_FIRST_BIT(v->u.rail.track)][v->direction]; + return TrackDirectionToTrackdir(FIND_FIRST_BIT(v->u.rail.track),v->direction); break; case VEH_Ship: if (v->u.ship.state == 0x80) /* Inside a depot? */ /* We'll assume the ship is facing outwards */ - return _dir_to_diag_trackdir[GetDepotDirection(v->tile, TRANSPORT_WATER)]; /* Ship in depot */ + return DiagdirToDiagTrackdir(GetDepotDirection(v->tile, TRANSPORT_WATER)); /* Ship in depot */ - return _track_direction_to_trackdir[FIND_FIRST_BIT(v->u.ship.state)][v->direction]; + return TrackDirectionToTrackdir(FIND_FIRST_BIT(v->u.ship.state),v->direction); break; case VEH_Road: if (v->u.road.state == 254) /* We'll assume the road vehicle is facing outwards */ - return _dir_to_diag_trackdir[GetDepotDirection(v->tile, TRANSPORT_ROAD)]; /* Road vehicle in depot */ + return DiagdirToDiagTrackdir(GetDepotDirection(v->tile, TRANSPORT_ROAD)); /* Road vehicle in depot */ if (IsRoadStationTile(v->tile)) /* We'll assume the road vehicle is facing outwards */ - return _dir_to_diag_trackdir[GetRoadStationDir(v->tile)]; /* Road vehicle in a station */ + return DiagdirToDiagTrackdir(GetRoadStationDir(v->tile)); /* Road vehicle in a station */ - return _dir_to_diag_trackdir[(v->direction >> 1) & 3]; + return DiagdirToDiagTrackdir((v->direction >> 1) & 3); break; /* case VEH_Aircraft: case VEH_Special: case VEH_Disaster: */ default: return 0xFF; diff --git a/vehicle.h b/vehicle.h index cee8a2450..5cd0f9db5 100644 --- a/vehicle.h +++ b/vehicle.h @@ -3,6 +3,7 @@ #include "pool.h" #include "order.h" +#include "rail.h" enum { VEH_Train = 0x10, @@ -340,7 +341,7 @@ typedef struct GetNewVehiclePosResult { * For other vehicles types, or vehicles with no clear trackdir (such as those * in depots), returns 0xFF. */ -byte GetVehicleTrackdir(const Vehicle* v); +Trackdir GetVehicleTrackdir(const Vehicle* v); /* returns true if staying in the same tile */ bool GetNewVehiclePos(Vehicle *v, GetNewVehiclePosResult *gp); -- cgit v1.2.3-54-g00ecf