#44688: Macrofy action_has_result() Open Date: 2022-05-27 10:34 Last Update: 2022-06-19 21:36 URL for this Ticket: https://osdn.net//projects/freeciv/ticket/44688 RSS feed for this Ticket: https://osdn.net/ticket/ticket_rss.php?group_id=12505&tid=44688 --------------------------------------------------------------------- Last Changes/Comment on this Ticket: 2022-06-19 21:36 Updated by: cazfi * Owner Update from (None) to cazfi * Resolution Update from None to Accepted * Milestone Update from (None) to 3.0.3 --------------------------------------------------------------------- Ticket Status: Reporter: cazfi Owner: cazfi Type: Patches Status: Open [Owner assigned] Priority: 5 - Medium MileStone: 3.0.3 Component: General Severity: 5 - Medium Resolution: Accepted --------------------------------------------------------------------- Ticket details: Fresh profiling of S3_1 server shows two new functions high in the list: action_by_number() with 11.37% share of processor time, and action_has_result() with 7.30%. action_has_result() seems like we should drop the asserts, and macrofy the remaining one-liner. -- 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/44688 RSS feed for this Ticket: https://osdn.net/ticket/ticket_rss.php?group_id=12505&tid=44688