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:


  Message sent via/by Gna!

Freeciv-dev mailing list

Reply via email to