I'm not sure that this operator is correct:

        bool operator<(const SoldierLevelRange& other) const {
                return max_health < other.min_health &&
                       max_attack < other.min_attack &&
                       max_defense < other.min_defense &&
                       max_evade < other.min_evade;
        }

For example, what if the range is identical except for health?
-- 
https://code.launchpad.net/~widelands-dev/widelands/per-level-soldier-anims/+merge/354929
Your team Widelands Developers is requested to review the proposed merge of 
lp:~widelands-dev/widelands/per-level-soldier-anims into lp:widelands.

_______________________________________________
Mailing list: https://launchpad.net/~widelands-dev
Post to     : widelands-dev@lists.launchpad.net
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp

Reply via email to