Follow-up Comment #2, patch #4445 (project freeciv): > Adds several strings, so not going to commit before 2.4.2 unless > jtn gives explicit ok. No, let's put this off until after 2.4.2.
Other nitpicks: I think this patch is mistakenly using AR_OK (airlift enum) instead of ATT_OK in some places. (No bug because they happen to have the same value.) String polish: "You can't attack there since there's unreachable unit.": suggest ... "there's *an* unreachable unit." _______________________________________________________ Reply to this item at: <http://gna.org/patch/?4445> _______________________________________________ Message sent via/by Gna! http://gna.org/ _______________________________________________ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev