Ticket #46544

Infinite recursion: found_new_tech() -> choose_tech() -> tech_researched() -> repeat

오픈 날짜: 2023-01-19 01:05 마지막 업데이트: 2023-03-20 03:31

Reporter:
소유자:
Type:
Status:
Closed
Component:
MileStone:
Priority:
5 - Medium
Severity:
5 - Medium
Resolution:
Fixed
File:
1

Details

Backtrace of a crashed S3_0 autogame reveals recursive loop of found_new_tech() -> choose_tech() -> tech_researched() -> from the beginning

choose_tech() seems to have 201 (which is A_FUTURE, I think) as a parameter.

Ticket History (3/7 Histories)

2023-01-19 01:05 Updated by: cazfi
  • New Ticket "Infinite recursion: found_new_tech() -> choose_tech() -> tech_researched() -> repeat" created
2023-01-19 02:35 Updated by: cazfi
댓글 올리기

This was caused by a local (-> not yet in repo) patch #45115.

A_FUTURE is a tech has it's requirements set to A_NEVER, yet it's supposed to be in game.

2023-01-19 06:43 Updated by: cazfi
댓글 올리기

Reply To cazfi

This was caused by a local (-> not yet in repo) patch #45115.

It's possible that A_FUTURE not being considered valid advance causes some other issues, but those would have been there for a long time, not to be a recent regression. As changing it has significant risk of breaking some caller, let's leave this to the next full cycle.

2023-02-14 13:38 Updated by: cazfi
  • 소유자 Update from (None) to cazfi
  • Resolution Update from None to Accepted
2023-02-17 13:38 Updated by: cazfi
  • Status Update from Open to Closed
  • Resolution Update from Accepted to Fixed
2023-03-20 03:31 Updated by: cazfi
댓글 올리기

Reply To cazfi

As changing it has significant risk of breaking some caller, let's leave this to the next full cycle.

We now have such a breakage in #47574

Edit

Please login to add comment to this ticket » Login