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