diff options
author | peter1138 <peter1138@openttd.org> | 2007-03-13 21:31:09 +0000 |
---|---|---|
committer | peter1138 <peter1138@openttd.org> | 2007-03-13 21:31:09 +0000 |
commit | 4177215b4111dedba3f4c7ce354a0b19d7edbf28 (patch) | |
tree | fe4d9d074e74884136ae01d5711b69fe05d33a72 | |
parent | c7bcdb69b185b239852ef6fa57aeeda5e86ceb3e (diff) | |
download | openttd-4177215b4111dedba3f4c7ce354a0b19d7edbf28.tar.xz |
(svn r9147) -Fix (r1770, r8161): when loadin games, enroute_from and enroute_from_xy were updated in the wrong place, causing issues with TTD savegames/scenarios.
-rw-r--r-- | src/openttd.cpp | 19 | ||||
-rw-r--r-- | src/station_cmd.cpp | 12 |
2 files changed, 19 insertions, 12 deletions
diff --git a/src/openttd.cpp b/src/openttd.cpp index 0452d6ab8..5a9c7499c 100644 --- a/src/openttd.cpp +++ b/src/openttd.cpp @@ -1792,6 +1792,25 @@ bool AfterLoadGame() FOR_ALL_VEHICLES(v) { v->cargo_source_xy = IsValidStationID(v->cargo_source) ? GetStation(v->cargo_source)->xy : v->tile; } + + /* Store position of the station where the goods come from, so there + * are no very high payments when stations get removed. However, if the + * station where the goods came from is already removed, the source + * information is lost. In that case we set it to the position of this + * station */ + Station *st; + FOR_ALL_STATIONS(st) { + for (CargoID c = 0; c < NUM_CARGO; c++) { + GoodsEntry *ge = &st->goods[c]; + + /* In old versions, enroute_from used 0xFF as INVALID_STATION */ + if (CheckSavegameVersion(7) && ge->enroute_from == 0xFF) { + ge->enroute_from = INVALID_STATION; + } + + ge->enroute_from_xy = IsValidStationID(ge->enroute_from) ? GetStation(ge->enroute_from)->xy : st->xy; + } + } } if (CheckSavegameVersion(45)) { diff --git a/src/station_cmd.cpp b/src/station_cmd.cpp index dae12547a..d62b609ed 100644 --- a/src/station_cmd.cpp +++ b/src/station_cmd.cpp @@ -2797,18 +2797,6 @@ static void SaveLoad_STNS(Station *st) SlObject(st, _station_desc); for (uint i = 0; i != NUM_CARGO; i++) { SlObject(&st->goods[i], _goods_desc); - - /* In older versions, enroute_from had 0xFF as INVALID_STATION, is now 0xFFFF */ - if (CheckSavegameVersion(7) && st->goods[i].enroute_from == 0xFF) { - st->goods[i].enroute_from = INVALID_STATION; - } - if (CheckSavegameVersion(44)) { - /* Store position of the station where the goods come from, so there are no - * very high payments when stations get removed. However, if the station - * where the goods came from is already removed, the source information is - * lost. In that case we set it to the position of this station */ - st->goods[i].enroute_from_xy = IsValidStationID(st->goods[i].enroute_from) ? GetStation(st->goods[i].enroute_from)->xy : st->xy; - } } if (st->num_specs != 0) { |