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 Freecivemail@example.com https://mail.gna.org/listinfo/freeciv-dev