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

_______________________________________________
Mailing list: https://launchpad.net/~widelands-dev
Post to     : [email protected]
Unsubscribe : https://launchpad.net/~widelands-dev
More help   : https://help.launchpad.net/ListHelp

Reply via email to