summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorbelugas <belugas@openttd.org>2006-04-14 01:54:07 +0000
committerbelugas <belugas@openttd.org>2006-04-14 01:54:07 +0000
commite84d878213e68263048befcaa8195344f683131f (patch)
tree666666f62052b4b05907ccbefd40e8b106d9bd85
parent8bff288817df4704461ee9c2f37969842c8e71ad (diff)
downloadopenttd-e84d878213e68263048befcaa8195344f683131f.tar.xz
(svn r4411) CodeChange : Define and use some Gfx for both stations and industries. More are still to come
Also, a bit of code cleaning, as pointed by Tron
-rw-r--r--industry_cmd.c20
-rw-r--r--industry_map.h10
-rw-r--r--station_cmd.c43
-rw-r--r--station_map.h85
4 files changed, 90 insertions, 68 deletions
diff --git a/industry_cmd.c b/industry_cmd.c
index e564ecbda..04a11f398 100644
--- a/industry_cmd.c
+++ b/industry_cmd.c
@@ -270,8 +270,8 @@ IndustryType GetIndustryType(TileIndex tile)
assert(IsTileType(tile, MP_INDUSTRY));
for (iloop = IT_COAL_MINE; iloop < IT_END; iloop += 1) {
- if IS_INT_INSIDE(this_type, industry_gfx_Solver[iloop].MinGfx,
- industry_gfx_Solver[iloop].MaxGfx) {
+ if (IS_BYTE_INSIDE(this_type, industry_gfx_Solver[iloop].MinGfx,
+ industry_gfx_Solver[iloop].MaxGfx)) {
return iloop;
}
}
@@ -531,7 +531,7 @@ static void AnimateTile_Industry(TileIndex tile)
byte m;
switch (GetIndustryGfx(tile)) {
- case 174:
+ case GFX_SUGAR_MINE_SIEVE:
if ((_tick_counter & 1) == 0) {
m = _m[tile].m3 + 1;
@@ -550,7 +550,7 @@ static void AnimateTile_Industry(TileIndex tile)
}
break;
- case 165:
+ case GFX_TOFFEE_QUARY:
if ((_tick_counter & 3) == 0) {
m = _m[tile].m3;
@@ -568,7 +568,7 @@ static void AnimateTile_Industry(TileIndex tile)
}
break;
- case 162:
+ case GFX_BUBBLE_CATCHER:
if ((_tick_counter&1) == 0) {
m = _m[tile].m3;
@@ -583,7 +583,7 @@ static void AnimateTile_Industry(TileIndex tile)
break;
// Sparks on a coal plant
- case 10:
+ case GFX_POWERPLANT_SPARKS:
if ((_tick_counter & 3) == 0) {
m = _m[tile].m1;
if (GB(m, 2, 5) == 6) {
@@ -596,7 +596,7 @@ static void AnimateTile_Industry(TileIndex tile)
}
break;
- case 143:
+ case GFX_TOY_FACTORY:
if ((_tick_counter & 1) == 0) {
m = _m[tile].m3 + 1;
@@ -733,9 +733,9 @@ static void MakeIndustryTileBigger(TileIndex tile)
if (GetIndustryGfx(tile + TileDiffXY(0, 1)) == 24) BuildOilRig(tile);
break;
- case 143:
- case 162:
- case 165:
+ case GFX_TOY_FACTORY:
+ case GFX_BUBBLE_CATCHER:
+ case GFX_TOFFEE_QUARY:
_m[tile].m3 = 0;
SetIndustryAnimationLoop(tile, 0);
break;
diff --git a/industry_map.h b/industry_map.h
index c31ef4087..a4bf06eef 100644
--- a/industry_map.h
+++ b/industry_map.h
@@ -9,7 +9,15 @@
#include "macros.h"
#include "tile.h"
-typedef uint IndustryGfx;
+typedef byte IndustryGfx;
+
+enum {
+ GFX_POWERPLANT_SPARKS = 10,
+ GFX_BUBBLE_CATCHER = 162,
+ GFX_TOFFEE_QUARY = 165,
+ GFX_SUGAR_MINE_SIEVE = 174,
+ GFX_TOY_FACTORY = 143
+};
static inline uint GetIndustryIndex(TileIndex t)
{
diff --git a/station_cmd.c b/station_cmd.c
index 4ae6e5533..5d3974bac 100644
--- a/station_cmd.c
+++ b/station_cmd.c
@@ -2067,15 +2067,15 @@ static void TileLoop_Station(TileIndex tile)
// FIXME -- GetTileTrackStatus_Station -> animated stationtiles
// hardcoded.....not good
switch (GetStationGfx(tile)) {
- case 0x27: // large big airport
- case 0x3A: // flag small airport
- case 0x5A: // radar international airport
- case 0x66: // radar metropolitan airport
+ case GFX_RADAR_LARGE_FIRST:
+ case GFX_WINDSACK_FIRST : // for small airport
+ case GFX_RADAR_INTERNATIONAL_FIRST: // radar international airport
+ case GFX_RADAR_METROPOLITAN_FIRST: // radar metropolitan airport
AddAnimatedTile(tile);
break;
- case 0x4B: // oilrig (station part)
- case 0x52: // bouy
+ case GFX_OILRIG_BASE: //(station part)
+ case GFX_BUOY_BASE:
TileLoop_Water(tile);
break;
@@ -2086,39 +2086,44 @@ static void TileLoop_Station(TileIndex tile)
static void AnimateTile_Station(TileIndex tile)
{
- byte gfx = GetStationGfx(tile);
+ StationGfx gfx = GetStationGfx(tile);
//FIXME -- AnimateTile_Station -> not nice code, lots of things double
- // again hardcoded...was a quick hack
+ // again hardcoded...was a quick hack
- // turning radar / windsack on airport
- if (gfx >= 39 && gfx <= 50) { // turning radar (39 - 50)
+ // turning radar / windsack on airport
+ if (IS_BYTE_INSIDE(gfx, GFX_RADAR_LARGE_FIRST, GFX_RADAR_LARGE_LAST)) {
if (_tick_counter & 3)
return;
- if (++gfx == 50+1)
- gfx = 39;
+ if (++gfx == GFX_RADAR_LARGE_LAST+1)
+ gfx = GFX_RADAR_LARGE_FIRST;
SetStationGfx(tile, gfx);
MarkTileDirtyByTile(tile);
- //added - begin
- } else if (gfx >= 90 && gfx <= 113) { // turning radar with ground under it (different fences) (90 - 101 | 102 - 113)
+ //added - begin
+ } else if (IS_BYTE_INSIDE(gfx, GFX_RADAR_INTERNATIONAL_FIRST, GFX_RADAR_METROPOLITAN_LAST)) {
if (_tick_counter & 3)
return;
gfx++;
- if (gfx == 101+1) {gfx = 90;} // radar with fences in south
- else if (gfx == 113+1) {gfx = 102;} // radar with fences in north
+ if (gfx == GFX_RADAR_INTERNATIONAL_LAST+1) {
+ gfx = GFX_RADAR_INTERNATIONAL_FIRST;
+ }
+ else if (gfx == GFX_RADAR_METROPOLITAN_LAST+1) {
+ gfx = GFX_RADAR_METROPOLITAN_FIRST;
+ }
SetStationGfx(tile, gfx);
MarkTileDirtyByTile(tile);
//added - end
- } else if (gfx >= 0x3A && gfx <= 0x3D) { // windsack (58 - 61)
+ } else if (IS_BYTE_INSIDE(gfx, GFX_WINDSACK_FIRST, GFX_WINDSACK_LAST)) {
if (_tick_counter & 1)
return;
- if (++gfx == 0x3D+1)
- gfx = 0x3A;
+ if (++gfx == GFX_WINDSACK_LAST+1) {
+ gfx = GFX_WINDSACK_FIRST;
+ }
SetStationGfx(tile, gfx);
MarkTileDirtyByTile(tile);
diff --git a/station_map.h b/station_map.h
index 0aed8b432..8e07fd49e 100644
--- a/station_map.h
+++ b/station_map.h
@@ -5,6 +5,8 @@
#include "station.h"
+typedef byte StationGfx;
+
static inline StationID GetStationIndex(TileIndex t)
{
assert(IsTileType(t, MP_STATION));
@@ -18,26 +20,33 @@ static inline Station* GetStationByTile(TileIndex t)
enum {
- RAILWAY_BASE = 0x0,
- AIRPORT_BASE = 0x8,
- TRUCK_BASE = 0x43,
- BUS_BASE = 0x47,
- OILRIG_BASE = 0x4B,
- DOCK_BASE = 0x4C,
- DOCK_BASE_WATER_PART = 0x50,
- BUOY_BASE = 0x52,
- AIRPORT_BASE_EXTENDED = 0x53,
-
- BASE_END = 0x73
+ GFX_RAILWAY_BASE = 0,
+ GFX_AIRPORT_BASE = 8,
+ GFX_RADAR_LARGE_FIRST = 39,
+ GFX_RADAR_LARGE_LAST = 50,
+ GFX_WINDSACK_FIRST = 58,
+ GFX_WINDSACK_LAST = 61,
+ GFX_TRUCK_BASE = 67,
+ GFX_BUS_BASE = 71,
+ GFX_OILRIG_BASE = 75,
+ GFX_DOCK_BASE = 76,
+ GFX_DOCK_BASE_WATER_PART = 80,
+ GFX_BUOY_BASE = 82,
+ GFX_AIRPORT_BASE_EXTENDED = 83,
+ GFX_RADAR_INTERNATIONAL_FIRST = 90,
+ GFX_RADAR_INTERNATIONAL_LAST = 101,
+ GFX_RADAR_METROPOLITAN_FIRST = 102,
+ GFX_RADAR_METROPOLITAN_LAST = 113,
+ GFX_BASE_END = 155
};
enum {
- RAILWAY_SIZE = AIRPORT_BASE - RAILWAY_BASE,
- AIRPORT_SIZE = TRUCK_BASE - AIRPORT_BASE,
- TRUCK_SIZE = BUS_BASE - TRUCK_BASE,
- BUS_SIZE = OILRIG_BASE - BUS_BASE,
- DOCK_SIZE_TOTAL = BUOY_BASE - DOCK_BASE,
- AIRPORT_SIZE_EXTENDED = BASE_END - AIRPORT_BASE_EXTENDED
+ RAILWAY_SIZE = GFX_AIRPORT_BASE - GFX_RAILWAY_BASE,
+ AIRPORT_SIZE = GFX_TRUCK_BASE - GFX_AIRPORT_BASE,
+ TRUCK_SIZE = GFX_BUS_BASE - GFX_TRUCK_BASE,
+ BUS_SIZE = GFX_OILRIG_BASE - GFX_BUS_BASE,
+ DOCK_SIZE_TOTAL = GFX_BUOY_BASE - GFX_DOCK_BASE,
+ AIRPORT_SIZE_EXTENDED = GFX_BASE_END - GFX_AIRPORT_BASE_EXTENDED
};
typedef enum HangarTiles {
@@ -65,13 +74,13 @@ static inline RoadStopType GetRoadStopType(TileIndex t)
return GetStationType(t) == STATION_TRUCK ? RS_TRUCK : RS_BUS;
}
-static inline byte GetStationGfx(TileIndex t)
+static inline StationGfx GetStationGfx(TileIndex t)
{
assert(IsTileType(t, MP_STATION));
return _m[t].m5;
}
-static inline void SetStationGfx(TileIndex t, byte gfx)
+static inline void SetStationGfx(TileIndex t, StationGfx gfx)
{
assert(IsTileType(t, MP_STATION));
_m[t].m5 = gfx;
@@ -79,7 +88,7 @@ static inline void SetStationGfx(TileIndex t, byte gfx)
static inline bool IsRailwayStation(TileIndex t)
{
- return GetStationGfx(t) < RAILWAY_BASE + RAILWAY_SIZE;
+ return GetStationGfx(t) < GFX_RAILWAY_BASE + RAILWAY_SIZE;
}
static inline bool IsRailwayStationTile(TileIndex t)
@@ -89,7 +98,7 @@ static inline bool IsRailwayStationTile(TileIndex t)
static inline bool IsHangar(TileIndex t)
{
- byte gfx = GetStationGfx(t);
+ StationGfx gfx = GetStationGfx(t);
return
gfx == HANGAR_TILE_0 ||
gfx == HANGAR_TILE_1 ||
@@ -98,20 +107,20 @@ static inline bool IsHangar(TileIndex t)
static inline bool IsAirport(TileIndex t)
{
- byte gfx = GetStationGfx(t);
+ StationGfx gfx = GetStationGfx(t);
return
- IS_INT_INSIDE(gfx, AIRPORT_BASE, AIRPORT_BASE + AIRPORT_SIZE) ||
- IS_INT_INSIDE(gfx, AIRPORT_BASE_EXTENDED, AIRPORT_BASE_EXTENDED + AIRPORT_SIZE_EXTENDED);
+ (IS_BYTE_INSIDE(gfx, GFX_AIRPORT_BASE, GFX_AIRPORT_BASE + AIRPORT_SIZE)) ||
+ (IS_BYTE_INSIDE(gfx, GFX_AIRPORT_BASE_EXTENDED, GFX_AIRPORT_BASE_EXTENDED + AIRPORT_SIZE_EXTENDED));
}
static inline bool IsTruckStop(TileIndex t)
{
- return IS_INT_INSIDE(GetStationGfx(t), TRUCK_BASE, TRUCK_BASE + TRUCK_SIZE);
+ return IS_BYTE_INSIDE(GetStationGfx(t), GFX_TRUCK_BASE, GFX_TRUCK_BASE + TRUCK_SIZE);
}
static inline bool IsBusStop(TileIndex t)
{
- return IS_INT_INSIDE(GetStationGfx(t), BUS_BASE, BUS_BASE + BUS_SIZE);
+ return IS_BYTE_INSIDE(GetStationGfx(t), GFX_BUS_BASE, GFX_BUS_BASE + BUS_SIZE);
}
static inline bool IsRoadStop(TileIndex t)
@@ -130,22 +139,22 @@ static inline bool IsRoadStopTile(TileIndex t)
static inline DiagDirection GetRoadStopDir(TileIndex t)
{
assert(IsRoadStopTile(t));
- return (GetStationGfx(t) - TRUCK_BASE) & 3;
+ return (GetStationGfx(t) - GFX_TRUCK_BASE) & 3;
}
static inline bool IsOilRig(TileIndex t)
{
- return GetStationGfx(t) == OILRIG_BASE;
+ return GetStationGfx(t) == GFX_OILRIG_BASE;
}
static inline bool IsDock(TileIndex t)
{
- return IS_INT_INSIDE(GetStationGfx(t), DOCK_BASE, DOCK_BASE + DOCK_SIZE_TOTAL);
+ return IS_BYTE_INSIDE(GetStationGfx(t), GFX_DOCK_BASE, GFX_DOCK_BASE + DOCK_SIZE_TOTAL);
}
static inline bool IsBuoy_(TileIndex t) // XXX _ due to naming conflict
{
- return GetStationGfx(t) == BUOY_BASE;
+ return GetStationGfx(t) == GFX_BUOY_BASE;
}
static inline bool IsBuoyTile(TileIndex t)
@@ -184,9 +193,9 @@ static inline bool IsCompatibleTrainStationTile(TileIndex t1, TileIndex t2)
static inline DiagDirection GetDockDirection(TileIndex t)
{
- byte gfx = GetStationGfx(t);
- assert(gfx < DOCK_BASE_WATER_PART);
- return (DiagDirection)(gfx - DOCK_BASE);
+ StationGfx gfx = GetStationGfx(t);
+ assert(gfx < GFX_DOCK_BASE_WATER_PART);
+ return (DiagDirection)(gfx - GFX_DOCK_BASE);
}
static inline TileIndexDiffC GetDockOffset(TileIndex t)
@@ -246,7 +255,7 @@ static inline void MakeRailStation(TileIndex t, Owner o, StationID sid, Axis a,
static inline void MakeRoadStop(TileIndex t, Owner o, StationID sid, RoadStopType rst, DiagDirection d)
{
- MakeStation(t, o, sid, (rst == RS_BUS ? BUS_BASE : TRUCK_BASE) + d);
+ MakeStation(t, o, sid, (rst == RS_BUS ? GFX_BUS_BASE : GFX_TRUCK_BASE) + d);
}
static inline void MakeAirport(TileIndex t, Owner o, StationID sid, byte section)
@@ -256,18 +265,18 @@ static inline void MakeAirport(TileIndex t, Owner o, StationID sid, byte section
static inline void MakeBuoy(TileIndex t, StationID sid)
{
- MakeStation(t, OWNER_NONE, sid, BUOY_BASE);
+ MakeStation(t, OWNER_NONE, sid, GFX_BUOY_BASE);
}
static inline void MakeDock(TileIndex t, Owner o, StationID sid, DiagDirection d)
{
- MakeStation(t, o, sid, DOCK_BASE + d);
- MakeStation(t + TileOffsByDir(d), o, sid, DOCK_BASE_WATER_PART + DiagDirToAxis(d));
+ MakeStation(t, o, sid, GFX_DOCK_BASE + d);
+ MakeStation(t + TileOffsByDir(d), o, sid, GFX_DOCK_BASE_WATER_PART + DiagDirToAxis(d));
}
static inline void MakeOilrig(TileIndex t, StationID sid)
{
- MakeStation(t, OWNER_NONE, sid, OILRIG_BASE);
+ MakeStation(t, OWNER_NONE, sid, GFX_OILRIG_BASE);
}
#endif