As currently implemented, this patch is huge step backwards in
relation to gen-movement. It even restores some functions we have
worked so hard to get rid of. It assumes a lot of unit class related
things to always be like they are in default ruleset (checking
is_ground_unit() & co instead of actual unit class properties). My
stealthy super spies are considered submarines :-)
 I've not spent time on detailed analysis if, and how, these issues
can be corrected. It seems obvious starting point that layer unit
belongs to is part of unit class definition.

 Good thing is that this is implemented as ruleset option. If above
issues can be solved, I'm all for taking this option to game.

 - ML

Freeciv-dev mailing list

Reply via email to