Review: Approve

Thanks for your clarifications. I thought we could fix that by consuming twice 
the wares, reduce the amount of coal by twice and produce twice the wares. But 
that has other implications (i.e. you need to have twice the wine before 
anything is produced). So I think your approach is reasonable.

train_workers: use a for (blah : m_training_positions) {} loop here if possible.

Otherwise lgtm.
-- 
https://code.launchpad.net/~widelands-dev/widelands/feature-1478-stone-mine-streamline/+merge/215594
Your team Widelands Developers is subscribed to branch 
lp:~widelands-dev/widelands/feature-1478-stone-mine-streamline.

_______________________________________________
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