From 49570edfe97df43654759e9d9e492137c0fe4c2a Mon Sep 17 00:00:00 2001 From: belugas Date: Fri, 18 May 2007 17:55:07 +0000 Subject: (svn r9870) -Codechange: Silence two compiler warnings and give proper type to the "type" member of industry struct --- src/economy.cpp | 5 +++-- src/industry.h | 2 +- src/industry_cmd.cpp | 4 ++-- 3 files changed, 6 insertions(+), 5 deletions(-) (limited to 'src') 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; } /** -- cgit v1.2.3-54-g00ecf