diff options
author | planetmaker <planetmaker@openttd.org> | 2011-10-05 17:39:12 +0000 |
---|---|---|
committer | planetmaker <planetmaker@openttd.org> | 2011-10-05 17:39:12 +0000 |
commit | 824f17215e6667e3b6df523f75407a78e6136e88 (patch) | |
tree | 5bc6111ba93acf3b67e685381f35a7b3750c69b3 /src | |
parent | 162b98faa5a0a72241e3519c41e283cee3155523 (diff) | |
download | openttd-824f17215e6667e3b6df523f75407a78e6136e88.tar.xz |
(svn r23005) -Fix (r23004): Of course there's still the 16-sprite version for shore tiles... we should continue to handle it.
Diffstat (limited to 'src')
-rw-r--r-- | src/newgrf.cpp | 20 |
1 files changed, 15 insertions, 5 deletions
diff --git a/src/newgrf.cpp b/src/newgrf.cpp index 37dc2587b..39ae46669 100644 --- a/src/newgrf.cpp +++ b/src/newgrf.cpp @@ -5332,11 +5332,21 @@ static void GraphicsNew(ByteReader *buf) const Action5Type *action5_type = &_action5_types[type]; - /* Contrary to TTDP we allow always to specify too few sprites as we allow always an offset. - * Thus no check for the amount of sprites is needed (anymore). - * The only no-offset entry, shores, is handled above already. - * We'll assume that offsets are always allowed further down, thus assert on this */ - assert(action5_type->block_type == A5BLOCK_ALLOW_OFFSET); + /* Contrary to TTDP we allow always to specify too few sprites as we allow always an offset, + * except for the long version of the shore type: + * Ignore offset if not allowed */ + if ((action5_type->block_type != A5BLOCK_ALLOW_OFFSET) && (offset != 0)) { + grfmsg(1, "GraphicsNew: %s (type 0x%02X) do not allow an <offset> field. Ignoring offset.", action5_type->name, type); + offset = 0; + } + + /* Ignore action5 if too few sprites are specified. (for TTDP compatibility) + * This does not make sense, if <offset> is allowed */ + if ((action5_type->block_type == A5BLOCK_FIXED) && (num < action5_type->min_sprites)) { + grfmsg(1, "GraphicsNew: %s (type 0x%02X) count must be at least %d. Only %d were specified. Skipping.", action5_type->name, type, action5_type->min_sprites, num); + _cur.skip_sprites = num; + return; + } /* Load at most max_sprites sprites. Skip remaining sprites. (for compatibility with TTDP and future extentions) */ uint16 skip_num = SanitizeSpriteOffset(num, offset, action5_type->max_sprites, action5_type->name); |