From 21cbc7009fe70737882d412cf67f9508e0cfd6d2 Mon Sep 17 00:00:00 2001 From: belugas Date: Wed, 26 Sep 2007 02:15:00 +0000 Subject: [PATCH] (svn r11163) -Codechange: Verify that the IndustryTileOverrideManager skip the magic value of 0xFF when assigning a new tileID. This is really important, since the value is reserved for water checking. --- src/newgrf_commons.cpp | 2 +- src/newgrf_commons.h | 3 +++ 2 files changed, 4 insertions(+), 1 deletion(-) diff --git a/src/newgrf_commons.cpp b/src/newgrf_commons.cpp index 6ffd879e0d..173997d99c 100644 --- a/src/newgrf_commons.cpp +++ b/src/newgrf_commons.cpp @@ -105,7 +105,7 @@ uint16 OverrideManagerBase::AddEntityID(byte grf_local_id, uint32 grfid, byte su for (id = max_offset; id < max_new_entities; id++) { map = &mapping_ID[id]; - if (map->entity_id == 0 && map->grfid == 0) { + if (CheckValidNewID(id) && map->entity_id == 0 && map->grfid == 0) { map->entity_id = grf_local_id; map->grfid = grfid; map->substitute_id = substitute_id; diff --git a/src/newgrf_commons.h b/src/newgrf_commons.h index 25933117ea..51ebd93b87 100644 --- a/src/newgrf_commons.h +++ b/src/newgrf_commons.h @@ -33,6 +33,7 @@ protected: uint16 max_new_entities; ///< what is the amount of entities, old and new summed uint16 invalid_ID; ///< ID used to dected invalid entities; + virtual bool CheckValidNewID(uint16 testid) { return true; } public: EntityIDMapping *mapping_ID; ///< mapping of ids from grf files. Public out of convenience @@ -76,6 +77,8 @@ public: struct IndustryTileSpec; class IndustryTileOverrideManager : public OverrideManagerBase { +protected: + virtual bool CheckValidNewID(uint16 testid) { return testid != 0xFF; } public: IndustryTileOverrideManager(uint16 offset, uint16 maximum, uint16 invalid) : OverrideManagerBase(offset, maximum, invalid) {}