diff --git a/src/subsidy.cpp b/src/subsidy.cpp --- a/src/subsidy.cpp +++ b/src/subsidy.cpp @@ -82,11 +82,11 @@ std::pairname); switch (s->src_type) { - case ST_INDUSTRY: + case SourceType::Industry: reftype1 = NR_INDUSTRY; SetDParam(parameter_offset + 1, STR_INDUSTRY_NAME); break; - case ST_TOWN: + case SourceType::Town: reftype1 = NR_TOWN; SetDParam(parameter_offset + 1, STR_TOWN_NAME); break; @@ -95,11 +95,11 @@ std::pairsrc); switch (s->dst_type) { - case ST_INDUSTRY: + case SourceType::Industry: reftype2 = NR_INDUSTRY; SetDParam(parameter_offset + 4, STR_INDUSTRY_NAME); break; - case ST_TOWN: + case SourceType::Town: reftype2 = NR_TOWN; SetDParam(parameter_offset + 4, STR_TOWN_NAME); break; @@ -124,8 +124,8 @@ std::pairpart_of_subsidy |= flag; return; - case ST_TOWN: Town::Get(index)->cache.part_of_subsidy |= flag; return; + case SourceType::Industry: Industry::Get(index)->part_of_subsidy |= flag; return; + case SourceType::Town: Town::Get(index)->cache.part_of_subsidy |= flag; return; default: NOT_REACHED(); } } @@ -196,8 +196,8 @@ static bool CheckSubsidyDuplicate(CargoI */ static bool CheckSubsidyDistance(SourceType src_type, SourceID src, SourceType dst_type, SourceID dst) { - TileIndex tile_src = (src_type == ST_TOWN) ? Town::Get(src)->xy : Industry::Get(src)->location.tile; - TileIndex tile_dst = (dst_type == ST_TOWN) ? Town::Get(dst)->xy : Industry::Get(dst)->location.tile; + TileIndex tile_src = (src_type == SourceType::Town) ? Town::Get(src)->xy : Industry::Get(src)->location.tile; + TileIndex tile_dst = (dst_type == SourceType::Town) ? Town::Get(dst)->xy : Industry::Get(dst)->location.tile; return (DistanceManhattan(tile_src, tile_dst) <= SUBSIDY_MAX_DISTANCE); } @@ -250,20 +250,20 @@ CommandCost CmdCreateSubsidy(DoCommandFl if (cid >= NUM_CARGO || !::CargoSpec::Get(cid)->IsValid()) return CMD_ERROR; switch (src_type) { - case ST_TOWN: + case SourceType::Town: if (!Town::IsValidID(src)) return CMD_ERROR; break; - case ST_INDUSTRY: + case SourceType::Industry: if (!Industry::IsValidID(src)) return CMD_ERROR; break; default: return CMD_ERROR; } switch (dst_type) { - case ST_TOWN: + case SourceType::Town: if (!Town::IsValidID(dst)) return CMD_ERROR; break; - case ST_INDUSTRY: + case SourceType::Industry: if (!Industry::IsValidID(dst)) return CMD_ERROR; break; default: @@ -297,9 +297,9 @@ bool FindSubsidyPassengerRoute() } if (DistanceManhattan(src->xy, dst->xy) > SUBSIDY_MAX_DISTANCE) return false; - if (CheckSubsidyDuplicate(CT_PASSENGERS, ST_TOWN, src->index, ST_TOWN, dst->index)) return false; + if (CheckSubsidyDuplicate(CT_PASSENGERS, SourceType::Town, src->index, SourceType::Town, dst->index)) return false; - CreateSubsidy(CT_PASSENGERS, ST_TOWN, src->index, ST_TOWN, dst->index); + CreateSubsidy(CT_PASSENGERS, SourceType::Town, src->index, SourceType::Town, dst->index); return true; } @@ -315,7 +315,7 @@ bool FindSubsidyTownCargoRoute() { if (!Subsidy::CanAllocateItem()) return false; - SourceType src_type = ST_TOWN; + SourceType src_type = SourceType::Town; /* Select a random town. */ const Town *src_town = Town::GetRandom(); @@ -373,7 +373,7 @@ bool FindSubsidyIndustryCargoRoute() { if (!Subsidy::CanAllocateItem()) return false; - SourceType src_type = ST_INDUSTRY; + SourceType src_type = SourceType::Industry; /* Select a random industry. */ const Industry *src_ind = Industry::GetRandom(); @@ -424,11 +424,11 @@ bool FindSubsidyIndustryCargoRoute() bool FindSubsidyCargoDestination(CargoID cid, SourceType src_type, SourceID src) { /* Choose a random destination. */ - SourceType dst_type = Chance16(1, 2) ? ST_TOWN : ST_INDUSTRY; + SourceType dst_type = Chance16(1, 2) ? SourceType::Town : SourceType::Industry; SourceID dst; switch (dst_type) { - case ST_TOWN: { + case SourceType::Town: { /* Select a random town. */ const Town *dst_town = Town::GetRandom(); @@ -448,7 +448,7 @@ bool FindSubsidyCargoDestination(CargoID break; } - case ST_INDUSTRY: { + case SourceType::Industry: { /* Select a random industry. */ const Industry *dst_ind = Industry::GetRandom(); if (dst_ind == nullptr) return false; @@ -565,10 +565,10 @@ bool CheckSubsidised(CargoID cargo_type, /* If the source isn't subsidised, don't continue */ if (src == INVALID_SOURCE) return false; switch (src_type) { - case ST_INDUSTRY: + case SourceType::Industry: if (!(Industry::Get(src)->part_of_subsidy & POS_SRC)) return false; break; - case ST_TOWN: + case SourceType::Town: if (!(Town::Get(src)->cache.part_of_subsidy & POS_SRC)) return false; break; default: return false; @@ -580,7 +580,7 @@ bool CheckSubsidised(CargoID cargo_type, if (!st->rect.IsEmpty()) { for (const Subsidy *s : Subsidy::Iterate()) { /* Don't create the cache if there is no applicable subsidy with town as destination */ - if (s->dst_type != ST_TOWN) continue; + if (s->dst_type != SourceType::Town) continue; if (s->cargo_type != cargo_type || s->src_type != src_type || s->src != src) continue; if (s->IsAwarded() && s->awarded != company) continue; @@ -601,7 +601,7 @@ bool CheckSubsidised(CargoID cargo_type, for (Subsidy *s : Subsidy::Iterate()) { if (s->cargo_type == cargo_type && s->src_type == src_type && s->src == src && (!s->IsAwarded() || s->awarded == company)) { switch (s->dst_type) { - case ST_INDUSTRY: + case SourceType::Industry: for (const auto &i : st->industries_near) { if (s->dst == i.industry->index) { assert(i.industry->part_of_subsidy & POS_DST); @@ -610,7 +610,7 @@ bool CheckSubsidised(CargoID cargo_type, } } break; - case ST_TOWN: + case SourceType::Town: for (const Town *tp : towns_near) { if (s->dst == tp->index) { assert(tp->cache.part_of_subsidy & POS_DST);