Mercurial > hg > openttd
changeset 17049:fbd7cc9a81ab draft
(svn r21786) -Fix [FS#4398]: don't trust rail station width and height data stored by TTDPatch, it is invalid for stations wider or higher than 7
author | smatz <smatz@openttd.org> |
---|---|
date | Fri, 14 Jan 2011 16:20:25 +0000 |
parents | 61552b237550 |
children | 0d6fb8c23e11 |
files | src/saveload/afterload.cpp |
diffstat | 1 files changed, 18 insertions(+), 13 deletions(-) [+] |
line wrap: on
line diff
--- a/src/saveload/afterload.cpp +++ b/src/saveload/afterload.cpp @@ -469,22 +469,27 @@ _pause_mode &= ~PMB_PAUSED_NETWORK; } - /* in very old versions, size of train stations was stored differently */ + /* In very old versions, size of train stations was stored differently. + * They had swapped width and height if station was built along the Y axis. + * TTO and TTD used 3 bits for width/height, while OpenTTD used 4. + * Because the data stored by TTDPatch are unusable for rail stations > 7x7, + * recompute the width and height. Doing this unconditionally for all old + * savegames simplifies the code. */ if (IsSavegameVersionBefore(2)) { Station *st; FOR_ALL_STATIONS(st) { - if (st->train_station.tile != 0 && st->train_station.h == 0) { - uint n = _savegame_type == SGT_OTTD ? 4 : 3; // OTTD uses 4 bits per dimensions, TTD 3 bits - uint w = GB(st->train_station.w, n, n); - uint h = GB(st->train_station.w, 0, n); - - if (GetRailStationAxis(st->train_station.tile) != AXIS_X) Swap(w, h); - - st->train_station.w = w; - st->train_station.h = h; - - assert(GetStationIndex(st->train_station.tile + TileDiffXY(w - 1, h - 1)) == st->index); - } + st->train_station.w = st->train_station.h = 0; + } + for (TileIndex t = 0; t < map_size; t++) { + if (!IsTileType(t, MP_STATION)) continue; + if (_m[t].m5 > 7) continue; // is it a rail station tile? + st = Station::Get(_m[t].m2); + assert(st->train_station.tile != 0); + int dx = TileX(t) - TileX(st->train_station.tile); + int dy = TileY(t) - TileY(st->train_station.tile); + assert(dx >= 0 && dy >= 0); + st->train_station.w = max<uint>(st->train_station.w, dx + 1); + st->train_station.h = max<uint>(st->train_station.h, dy + 1); } }