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 Freeciv-dev@gna.org https://mail.gna.org/listinfo/freeciv-dev
