#44594: TECH_UPKEEP_DEBUGGING messing research->techs_researched when future techs involved Open Date: 2022-05-14 16:13 Last Update: 2022-05-28 21:55 URL for this Ticket: https://osdn.net//projects/freeciv/ticket/44594 RSS feed for this Ticket: https://osdn.net/ticket/ticket_rss.php?group_id=12505&tid=44594 --------------------------------------------------------------------- Last Changes/Comment on this Ticket: 2022-05-28 21:55 Updated by: cazfi * Milestone Update from 3.0.2 to 3.1.0 Comment: Maybe it's not a very good idea to introduce that sanity check to an active release branch. While it could help us to find the problems, we don't want it to start constantly failing on people as a regression compared to earlier 3.0 releases. People who want to test S3_0 for this problem, can use such sanity check locally. --------------------------------------------------------------------- Ticket Status: Reporter: cazfi Owner: (None) Type: Bugs Status: Open Priority: 5 - Medium MileStone: 3.1.0 Component: Server Severity: 5 - Medium Resolution: None --------------------------------------------------------------------- Ticket details: init_tech() recalculates research->techs_researched when TECH_UPKEEP_DEBUGGING is defined. That recalculation does not consider future techs. This is one candidate for the cause of various assert failures I'm getting in testing #44419. When number of lost (i.e. ones subtracted from the counter) future techs is same as number of real techs, the counter goes to zero despite player knowing a lot of techs. -- 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/44594 RSS feed for this Ticket: https://osdn.net/ticket/ticket_rss.php?group_id=12505&tid=44594