Follow-up Comment #2, bug #22400 (project freeciv):
New version attached. It ignores numbering. It merges the two if statements
that validated tech target.
_Your test looks a bit weak._
Correct. A new renumbering could have broken it.
_What would happen if the tech is A_LAST? Or if the tech is out of ruleset
The if statement below would have caught it. (They are merged in the new
patch. Easier to read)
Additional Item Attachment:
File name: 0001-Fix-spy-stealing-of-future-techs-and-of-techs-chosen.patch
Reply to this item at:
Message sent via/by Gna!
Freeciv-dev mailing list