Update of patch #3901 (project freeciv): Planned Release: => 2.6.0
_______________________________________________________ Follow-up Comment #4: After some discussion on IRC, pepeto and I concluded that the comment "/* Entering port or bombardment */" introduced into sea_overlap_move() in r12065 was, while sensible, not the intended purpose of this use of SINGLE_MOVE. Rather, this use of SINGLE_MOVE is intended to represent the cost of moving to normally inaccessible territory as a result of overlap movement, to facilitate path planning for ferries. As a result, it is probably inappropriate to check pf_is_able_to_attack() in overlap moves, and there should probably be a new function overlap_move_cost, initially always returning SINGLE_MOVE, but used to indicate semantic construction for future developers encountering this issue, or needing to apply special values for this condition. There was also some discussion about the appropriate ordering of the city_channel_tile() check and the potential_transport check in pf_is_ok_move_tile(), and interaction of this ordering with potential application of is_source_tile to city_channels and use in overlap moves. Note that bug #21871 is now targeted only towards S2_5, so this patch must also address that issue in final form. Review remains underway: those with rulesets that depend on complex nativity may feel free to test with the current patch (which addresses the vast majority of client-side GoTo issues), but be aware there may be additional changes as a result of deeper review. _______________________________________________________ Reply to this item at: <http://gna.org/patch/?3901> _______________________________________________ Message sent via/by Gna! http://gna.org/ _______________________________________________ Freeciv-dev mailing list Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev