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


Reply to this item at:


  Message sent via/by Gna!

Freeciv-dev mailing list

Reply via email to