diff options
author | belugas <belugas@openttd.org> | 2007-05-18 17:55:07 +0000 |
---|---|---|
committer | belugas <belugas@openttd.org> | 2007-05-18 17:55:07 +0000 |
commit | 662bee5b12192167a58423f9d8f5e2ce800f17e2 (patch) | |
tree | 4b779bb86f97ebb5f7398c295675a5e7140c1d29 | |
parent | c8d05162c296a6fdec7a02163b21e2fce2da7272 (diff) | |
download | openttd-662bee5b12192167a58423f9d8f5e2ce800f17e2.tar.xz |
(svn r9870) -Codechange: Silence two compiler warnings and give proper type to the "type" member of industry struct
-rw-r--r-- | src/economy.cpp | 5 | ||||
-rw-r--r-- | src/industry.h | 2 | ||||
-rw-r--r-- | src/industry_cmd.cpp | 4 |
3 files changed, 6 insertions, 5 deletions
diff --git a/src/economy.cpp b/src/economy.cpp index c68fe5672..d6f0e629c 100644 --- a/src/economy.cpp +++ b/src/economy.cpp @@ -1190,7 +1190,7 @@ static void DeliverGoodsToIndustry(TileIndex xy, CargoID cargo_type, int num_pie Industry *ind; const IndustrySpec *indspec; uint best_dist; - uint accepted_cargo_index; + uint accepted_cargo_index = 0; ///< unlikely value, just for warning removing /* Check if there's an industry close to the station that accepts the cargo * XXX - Think of something better to @@ -1199,10 +1199,10 @@ static void DeliverGoodsToIndustry(TileIndex xy, CargoID cargo_type, int num_pie best_dist = (_patches.station_spread + 8) * 2; FOR_ALL_INDUSTRIES(ind) { indspec = GetIndustrySpec(ind->type); + uint i; if (indspec->produced_cargo[0] == CT_INVALID) continue; - uint i; for (i = 0; i < lengthof(indspec->accepts_cargo); i++) { if (cargo_type == indspec->accepts_cargo[i] && (indspec->input_cargo_multiplier[i][0] != 0 || indspec->input_cargo_multiplier[i][1] != 0)) { @@ -1210,6 +1210,7 @@ static void DeliverGoodsToIndustry(TileIndex xy, CargoID cargo_type, int num_pie } } + /* Check if matching cargo has been found */ if (i == lengthof(indspec->accepts_cargo)) continue; uint dist = DistanceManhattan(ind->xy, xy); diff --git a/src/industry.h b/src/industry.h index 677d622e7..9f0ad8083 100644 --- a/src/industry.h +++ b/src/industry.h @@ -77,7 +77,7 @@ struct Industry { uint16 total_transported[2]; ///< total units transported per cargo uint16 counter; ///< used for animation and/or production (if available cargo) - byte type; ///< type of industry. see IT_COAL_MINE and others + IndustryType type; ///< type of industry. see IT_COAL_MINE and others OwnerByte owner; ///< owner of the industry. Which SHOULD always be (imho) OWNER_NONE byte random_color; ///< randomized colour of the industry, for display purpose Year last_prod_year; ///< last year of production diff --git a/src/industry_cmd.cpp b/src/industry_cmd.cpp index 1fa12d0fd..c30ed3d4e 100644 --- a/src/industry_cmd.cpp +++ b/src/industry_cmd.cpp @@ -63,8 +63,8 @@ IndustryType GetIndustryType(TileIndex tile) { assert(IsTileType(tile, MP_INDUSTRY)); - const Industry *ind = GetIndustry(GetIndustryIndex(tile)); - return IsValidIndustry(ind) ? ind->type : IT_INVALID; + const Industry *ind = GetIndustryByTile(tile); + return IsValidIndustry(ind) ? ind->type : (IndustryType)IT_INVALID; } /** |