Follow-up Comment #6, patch #4051 (project freeciv):

> Assuming this one goes in, I will be requesting follow-up patch to make
diplstate_text() to use names of the states from the diplstate_type specenum
instead of defining (identical) strings itself.
Just to be clear before I start writing it: Do you mean eliminating
diplstate_text() and change its users to use the specenum's functions in stead
or make diplstate_text() wrap?

> I would use something like "Hosts embassy" and "Has embassy"
Thank you for the suggestion. Changed in the new version.

The new version also renames diplrel_between() to diplrel_between_is() and
diplrel_to_other() to diplrel_to_other_is().

(file #18682)
    _______________________________________________________

Additional Item Attachment:

File name: diplrel.patch                  Size:17 KB


    _______________________________________________________

Reply to this item at:

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

_______________________________________________
  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