widelands-dev team mailing list archive
-
widelands-dev team
-
Mailing list archive
-
Message #07876
Re: [Merge] lp:~widelands-dev/widelands/bug-1603763 into lp:widelands
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.
References