#44345: Load counter from ruleset
Open Date: 2022-04-13 12:06
Last Update: 2022-06-27 02:37
URL for this Ticket:
https://osdn.net//projects/freeciv/ticket/44345
RSS feed for this Ticket:
https://osdn.net/ticket/ticket_rss.php?group_id=12505&tid=44345
---------------------------------------------------------------------
Last Changes/Comment on this Ticket:
2022-06-27 02:37 Updated by: lachu
Comment:
Reply To (Anonymous)
Reply To cazfi
Looks good. Just two minor style issues
- Prefer i++ instead of ++i when either works. That needs fixing in a couple of "for (... ; ... ; ++xxx)"
- Couple of "save_default_int(... path,"xxx")" lack the space after comma
While we ideally would want to keep the ruleset documentation in sync with the code, this is getting too slow. Maybe we should split this ticket to the code change (almost ready) and to the documentation update to be handled after this?
I think better approach is move to another target milestone/version. I have also other task and do not known how fast could I deliver changes :-( .
This comment was mine. I have foreign language's trouble, so writing documentation is not my strong side.
---------------------------------------------------------------------
Ticket Status:
Reporter: cazfi
Owner: (None)
Type: Patches
Status: Open
Priority: 5 - Medium
MileStone: 3.2.0
Component: General
Severity: 5 - Medium
Resolution: None
---------------------------------------------------------------------
Ticket details:
Originally: #41121
Replace internal City Owned counter definition by loading zero or one counter definitions from the ruleset. There's no need to send the info to client yet.
--
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/44345
RSS feed for this Ticket:
https://osdn.net/ticket/ticket_rss.php?group_id=12505&tid=44345