widelands-dev team mailing list archive

[SirVer <SirVer@xxxxxx>]

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.