HostedRedmine.com has moved to the Planio platform. All logins and passwords remained the same. All users will be able to login and use Redmine just as before. Read more...
Bug #850673
civ1 land unit class claims it can airlift
0%
Description
It can't.
History
#1
Updated by Sveinung Kvilhaugsvik about 1 year ago
- File 0001-civ1-remove-lie-from-land-unit-class.patch 0001-civ1-remove-lie-from-land-unit-class.patch added
- Status changed from In Progress to Resolved
#2
Updated by Marko Lindqvist about 1 year ago
Should we remove "Airliftable" unit class flag from civ1 land units in S2_6 too? It seems there is no helptext generated about it currently, so it's not having visible effects I think.
#3
Updated by Sveinung Kvilhaugsvik about 1 year ago
Marko Lindqvist wrote:
Should we remove "Airliftable" unit class flag from civ1 land units in S2_6 too? It seems there is no helptext generated about it currently, so it's not having visible effects I think.
Are you sure?
nef wrote in http://forum.freeciv.org/f/viewtopic.php?f=15&t=597&p=100875&#p100875
"Airliftable" flag in civ1 should be removed from [unitclass_land]. Can never be done because the effect Airlift is never set but AGH hasn't figured this out (and probably shouldn't need to).
#4
Updated by Marko Lindqvist about 1 year ago
- Sprint/Milestone changed from 3.0.0 to 2.6.1
Indeed. I had accidentally grepped helpdata.c for "Airliftable" that is ruleset author visible name of the flag and not UCF_AIRLIFTABLE that is C name for the flag.
So we really should remove the flag.
#6
Updated by Sveinung Kvilhaugsvik about 1 year ago
- Status changed from Resolved to Closed
.