#45889: Send initial counter values to client
Open Date: 2022-10-17 14:08
Last Update: 2022-12-20 12:07
URL for this Ticket:
https://osdn.net//projects/freeciv/ticket/45889
RSS feed for this Ticket:
https://osdn.net/ticket/ticket_rss.php?group_id=12505&tid=45889
---------------------------------------------------------------------
Last Changes/Comment on this Ticket:
2022-12-20 12:07 Updated by: cazfi
Comment:
Reply To lachu
should been send on sending city information.
That just seems obvious to me - city counter values *are* part of city related information, aren't they?
As for trying to figure out what counters to send on each situation, on some high level functions; 1) delta network protocol already automatically makes it so that not-changed values are not actually sent. 2) Implementing it on high level will only bring very complex code, with lots of bugs, high maintenance, and all the special case handling code making bigger NEGATIVE performance impact than what one would save in avoiding refreshing some field.
---------------------------------------------------------------------
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:
City counter value updates are sent to client by #45429, but we will need to send also initial values, e.g., when a client reconnects, or game has been loaded from a saved game. Basically when ever full city info gets sent to the client for the first time (note, that if it makes implementation easier; it should be ok to "send" them to the delta protocol layer unconditionally when ever full city info gets sent. Delta protocol would notice itself if there's nothing to update/send)
--
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/45889
RSS feed for this Ticket:
https://osdn.net/ticket/ticket_rss.php?group_id=12505&tid=45889