Update of patch #1975 (project freeciv):
Status: None => In Progress
Planned Release: => 2.3.0
Follow-up Comment #1:
When I read the content of this item, I was afraid it wouldn't be
configurable. As it is done by an option, this patch is good.
* There are tabs, they should be replaced by spaces (newly code).
* Passing F_NOHOME is not right here because it's a unit type flag, and not a
city id. It should be IDENTITY_NUMBER_ZERO, according to
create_unit_virtual(). Note that it's not the only place IDENTITY_NUMBER_ZERO
is missing to test unit::homecity. Probably it could be replaced by a pointer
Reply to this item at:
Message posté via/par Gna!
Freeciv-dev mailing list