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
Follow-up Comment #1, bug #22400 (project freeciv):
Your test looks a bit weak. What would happen if the tech is A_LAST? Or if the
tech is out of ruleset limits?
I think it shoould looks like:
(A_NONE == tech
|| (A_UNSET != tech
A_FUTURE != tech
NULL ==
URL:
http://gna.org/bugs/?22400
Summary: Stealing random and future tech don't work
Project: Freeciv
Submitted by: sveinung
Submitted on: Mon 28 Jul 2014 12:08:40 PM UTC
Category: None
Severity: 3 - Normal