[Freeciv-tickets] [freeciv] #44703: Optimize create_city()

Back to archive index
OSDN Ticket System norep****@osdn*****
Mon May 30 07:49:28 JST 2022


#44703: Optimize create_city()

  Open Date: 2022-05-29 00:25
Last Update: 2022-05-30 01:49

URL for this Ticket:
    https://osdn.net//projects/freeciv/ticket/44703
RSS feed for this Ticket:
    https://osdn.net/ticket/ticket_rss.php?group_id=12505&tid=44703

---------------------------------------------------------------------

Last Changes/Comment on this Ticket:
2022-05-30 01:49 Updated by: ihnatus

Comment:

Reply To cazfi
- "Set from UINT8 packet field, no need to check the top" sounds a bit like something to make a FC_STATIC_ASSERT() about, if possible
Could not find a way, UINT8 packet fields still have type int, put a usual check.
Here is a patch for master, I hope I understood how rscompat works. Do you think we should still emit "city_size_change" signals on city creation in 3.2?

---------------------------------------------------------------------
Ticket Status:

      Reporter: ihnatus
         Owner: (None)
          Type: Feature Requests
        Status: Open
      Priority: 5 - Medium
     MileStone: 3.0.2
     Component: Server
      Severity: 5 - Medium
    Resolution: None
---------------------------------------------------------------------

Ticket details:

Add size parameter to create_city()function. As a side effect, allow creation of cities bigger than maximal size an empty city may grow to (a notice is left). As another related fix, check city_size sanity for all ruleset units.

-- 
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/44703
RSS feed for this Ticket:
    https://osdn.net/ticket/ticket_rss.php?group_id=12505&tid=44703



More information about the Freeciv-tickets mailing list
Back to archive index