#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