#48786: specenum generator: Value prefix in header
Open Date: 2023-10-01 22:51
Last Update: 2023-10-02 11:07
URL for this Ticket:
https://osdn.net//projects/freeciv/ticket/48786
RSS feed for this Ticket:
https://osdn.net/ticket/ticket_rss.php?group_id=12505&tid=48786
---------------------------------------------------------------------
Last Changes/Comment on this Ticket:
2023-10-02 11:07 Updated by: alienvalkyrie
* Owner Update from (None) to alienvalkyrie
* Resolution Update from None to Accepted
Comment:
Attached patch. Using the prefix is optional, so it's backwards compatible with other enums converted to the new format in the interim (e.g. #48785).
I don't like that this now leads to writing count COUNT a bunch; will make a follow-up for that.
---------------------------------------------------------------------
Ticket Status:
Reporter: cazfi
Owner: alienvalkyrie
Type: Patches
Status: Open [Owner assigned]
Priority: 5 - Medium
MileStone: 3.3.0
Component: Bootstrap
Severity: 5 - Medium
Resolution: Accepted
---------------------------------------------------------------------
Ticket details:
Currently each value in specenum generator .def file contains the prefix, e.g., MANUAL_SETTINGS, MANUAL_COMMANDS ...
Should we move the prefix to header part, and define values simply as SETTINGS, COMMANDS ... ?
--
Ticket information of Freeciv project
Freeciv Project is hosted on OSDN
Project URL: https://osdn.net/projects/freeciv/
OSDN: https://osdn.net
URL for this Ticket:
https://osdn.net/projects/freeciv/ticket/48786
RSS feed for this Ticket:
https://osdn.net/ticket/ticket_rss.php?group_id=12505&tid=48786