#44852: Redundant check in citizens_update() Open Date: 2022-06-17 23:28 Last Update: 2022-06-17 23:28 URL for this Ticket: https://osdn.net//projects/freeciv/ticket/44852 RSS feed for this Ticket: https://osdn.net/ticket/ticket_rss.php?group_id=12505&tid=44852 --------------------------------------------------------------------- Last Changes/Comment on this Ticket: 2022-06-17 23:28 Updated by: ihnatus * New Ticket "Redundant check in citizens_update()" created --------------------------------------------------------------------- Ticket Status: Reporter: ihnatus Owner: (None) Type: Bugs Status: Open Priority: 5 - Medium MileStone: 3.0.3 Component: Server Severity: 4 Resolution: None --------------------------------------------------------------------- Ticket details: int selected = fc_rand(count); ... if (selected != count) is always true. That does not change the result (just sometimes we assign a thing to itself) but is redundant. -- 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/44852 RSS feed for this Ticket: https://osdn.net/ticket/ticket_rss.php?group_id=12505&tid=44852