Unless Sirver finds some nits to pick in this branch, I'd be fine with separate merges. This would keep the code reviews to smaller chunks.
The only thing that stands out for me here it that renaming built_mat_ to build_material_ will make the code more readable. -- https://code.launchpad.net/~widelands-dev/widelands/tibor-ai6/+merge/238767 Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/tibor-ai6 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
