Follow-up Comment #6, bug #15266 (project freeciv):
> But I think, IIRC, some parts of the code still assume borders
> are enabled. SO I think a such crash is possible also in S2_1.
> I want to check this before fixing this item.
You are right. I checked S2_1 and the following code is used:
void tile_set_owner(struct tile *ptile, struct player *owner)
{
if (0 < game.info.borders) {
ptile->owner = owner;
}
}
So for game info.borders == 0, the tile owner is not set. I have check both
versions, S2_1 and S_2_2 / trunk, again and will attach the updated patches.
Both compile but I did not test all possible cases ...
(file #7904, file #7905)
_______________________________________________________
Additional Item Attachment:
File name: S2_1-fix-assert-for-tile_owner-if-borders-0-S2_1.patch Size:4 KB
File name: S2_2-trunk-fix-assert-for-tile_owner-if-borders-0.patch Size:5 KB
_______________________________________________________
Reply to this item at:
<http://gna.org/bugs/?15266>
_______________________________________________
Nachricht geschickt von/durch Gna!
http://gna.org/
_______________________________________________
Freeciv-dev mailing list
[email protected]
https://mail.gna.org/listinfo/freeciv-dev
