Code LGTM, 2 nits. Needs testing.
Is there a reason you can't remove enum class OverlayLevel {} right now? If the answer is "yes", ignore this comment, otherwise remove. The roads that are displayed while a road is being build. -> built. Sorry for the review format, just a quick check-in during lunch break ;) -- https://code.launchpad.net/~widelands-dev/widelands/reduce_overlay_manager_use3/+merge/330496 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/reduce_overlay_manager_use3 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