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
limits?_
The if statement below would have caught it. (They are merged in the new
patch. Easier to read)
(file #21594)
_______________________________________________________
Additional Item Attachment:
File name: 0001-Fix-spy-stealing-of-future-techs-and-of-techs-chosen.patch
Size:1 KB
_______________________________________________________
Reply to this item at:
<http://gna.org/bugs/?22400>
_______________________________________________
Message sent via/by Gna!
http://gna.org/
_______________________________________________
Freeciv-dev mailing list
[email protected]
https://mail.gna.org/listinfo/freeciv-dev
