widelands-dev team mailing list archive

[SirVer <SirVer@xxxxxx>]

> Is there a reason you can't
remove enum class OverlayLevel {} right now? If the answer is "yes",
ignore this comment, otherwise remove.

No, it could be removed. but since it is part of the public API of the class it would entail further changes. Since the whole class is deleted in the next change after this one is merged (already done, just waiting for this to get in) I did not want to do the work.

> while a road is being build. -> built.

done. 
-- 
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.