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
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev

Reply via email to