Review: Approve I agree this is an improvement. One nit though: I think INVALID_INDEX == -1, but I cannot check. If that is the case, the first condition in the OR can be removed.
Otherwise lgtm. -- https://code.launchpad.net/~widelands-dev/widelands/bug-1603763/+merge/300278 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bug-1603763. _______________________________________________ Mailing list: https://launchpad.net/~widelands-dev Post to : [email protected] Unsubscribe : https://launchpad.net/~widelands-dev More help : https://help.launchpad.net/ListHelp
