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

More detailed review:

1) if/of typo (see comment #2)
2) Uses "minsize", rather than "minculture" in !preq->present (else)
conditions in helpdata.c
3) cityturn.c:: notification worklist_change_build_target(): text implies
REQ_RANGE_CITY (although this is generally true for this function, so perhaps
it isn't the responsibility of this patch to fix it).

    I don't see anything else, and this time I looked carefully, rather than
just encountering it because I'd applied the patch in a local branch.


    _______________________________________________________

Reply to this item at:

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

_______________________________________________
  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