Follow-up Comment #2, bug #17096 (project freeciv):
I'm not entirely sure I believe this assertion is valid.
I think v1/v2/v3 index (tileset_layer)->match_types[], which is a dynamically
allocated array with no hard limit. In particular, I don't think that it's
limited by (drawing_layer)->match_indices, which is what I think this
assertion is asserting.
In this case I'm guessing it's firing as a knock-on effect of
(drawing_layer)->match_indices having been truncated by the code that's fixed
by bug #17095, presumably to less than (tileset_layer)->match_count.
Perhaps it's the case that there can't _usefully_ be more match_types than
match_indices; I don't understand this matching stuff well enough to comment
on that. If so, the condition should be enforced by an assertion in
tileset_read_toplevel(), rather than leaving it to fall through to
tileset_setup_tile_type().
_______________________________________________________
Reply to this item at:
<http://gna.org/bugs/?17096>
_______________________________________________
Message sent via/by Gna!
http://gna.org/
_______________________________________________
Freeciv-dev mailing list
[email protected]
https://mail.gna.org/listinfo/freeciv-dev
