Review: Approve I am slightly confused. Why does it say this is merged? Trunk does definitely not contain the new code. Could it be that you branched and merged your different branches from each other instead from trunk?
however, the code looks fine. Just a nit upcast(Warehouse, wh, building); if (wh) can be written as if (upcast(Warehouse, wh, building)) -- https://code.launchpad.net/~widelands-dev/widelands/warehouse_fix/+merge/177279 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/warehouse_fix. _______________________________________________ Mailing list: https://launchpad.net/~widelands-dev Post to : [email protected] Unsubscribe : https://launchpad.net/~widelands-dev More help : https://help.launchpad.net/ListHelp
