Mercurial > hg > openttd
diff src/economy.cpp @ 10498:730117b172e1 draft
(svn r14753) -Fix (r1): after buying a company, one could have more vehicles with the same UnitID
author | smatz <smatz@openttd.org> |
---|---|
date | Fri, 26 Dec 2008 23:53:07 +0000 (2008-12-26) |
parents | dd7d08e0ff23 |
children | 45ca88a8de7d |
line wrap: on
line diff
--- a/src/economy.cpp +++ b/src/economy.cpp @@ -363,25 +363,12 @@ } { - int num_train = 0; - int num_road = 0; - int num_ship = 0; - int num_aircraft = 0; - Vehicle *v; + FreeUnitIDGenerator unitidgen[] = { + FreeUnitIDGenerator(VEH_TRAIN, new_owner), FreeUnitIDGenerator(VEH_ROAD, new_owner), + FreeUnitIDGenerator(VEH_SHIP, new_owner), FreeUnitIDGenerator(VEH_AIRCRAFT, new_owner) + }; - /* Determine Ids for the new vehicles */ - FOR_ALL_VEHICLES(v) { - if (v->owner == new_owner) { - switch (v->type) { - case VEH_TRAIN: if (IsFrontEngine(v)) num_train++; break; - case VEH_ROAD: if (IsRoadVehFront(v)) num_road++; break; - case VEH_SHIP: num_ship++; break; - case VEH_AIRCRAFT: if (IsNormalAircraft(v)) num_aircraft++; break; - default: break; - } - } - } - + Vehicle *v; FOR_ALL_VEHICLES(v) { if (v->owner == old_owner && IsCompanyBuildableVehicleType(v->type)) { if (new_owner == INVALID_OWNER) { @@ -391,13 +378,7 @@ v->colormap = PAL_NONE; v->group_id = DEFAULT_GROUP; if (IsEngineCountable(v)) GetCompany(new_owner)->num_engines[v->engine_type]++; - switch (v->type) { - case VEH_TRAIN: if (IsFrontEngine(v)) v->unitnumber = ++num_train; break; - case VEH_ROAD: if (IsRoadVehFront(v)) v->unitnumber = ++num_road; break; - case VEH_SHIP: v->unitnumber = ++num_ship; break; - case VEH_AIRCRAFT: if (IsNormalAircraft(v)) v->unitnumber = ++num_aircraft; break; - default: NOT_REACHED(); - } + if (v->IsPrimaryVehicle()) v->unitnumber = unitidgen[v->type].NextID(); } } }