Update of patch #4970 (project freeciv):
Status: Ready For Test = Done
Open/Closed:Open = Closed
___
Reply to this item at:
Follow-up Comment #3, patch #4970 (project freeciv):
I prefer symbols to magic numbers, but agree -1 isn't PF_IMPOSSIBLE_MC in this
case. Thanks for the explanation on attack cost. Everything else in this
patch looked good, so please consider me not to have objected at all.
URL:
http://gna.org/patch/?4970
Summary: Better path-finding reverse maps
Project: Freeciv
Submitted by: pepeto
Submitted on: lun. 21 juil. 2014 16:38:14 CEST
Category: general
Priority: 5 - Normal
Follow-up Comment #1, patch #4970 (project freeciv):
There are a couple surviving uses of -1 as a magic number in
pf_reverse_map_get_costs(): could these use PF_IMPOSSIBLE_MC?
Attacks are also still charged at SINGLE_MOVE: should this be increased for
UCF_MISSILE and UTYF_ONEATTACK, or don't we
