Follow-up Comment #2, patch #3447 (project freeciv):

This patch (transitional state it produces) has some chicken-and-egg design
problems with territory claiming bases. Though the version I finally submitted
avoided (in a hackish way - code recurses more than it needs to, but changes
between rounds make it to avoid infinite recursion) most obvious problems, I
now think that it doesn't work correctly when tile has multiple bases. When
the recursion causes other bases to change owner, tile is already owned by new
owner and as tile owner is in this transtional phase used as base owner, new
owner is considered old base owner -> base is not cleared from real old owner,
and as it seems like base is not changing hands at all, some base capturing
stuff is skipped too.

    _______________________________________________________

Reply to this item at:

  <http://gna.org/patch/?3447>

_______________________________________________
  Message sent via/by Gna!
  http://gna.org/


_______________________________________________
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev

Reply via email to