widelands-dev team mailing list archive

[GunChleoc <fios@xxxxxxxxxxxxxxxxxxx>]

Review: Needs Fixing

I have been fiddling with a test branch https://code.launchpad.net/~widelands-dev/widelands/dynamic_tribe_loading_test and I have found the change that broke it:

http://bazaar.launchpad.net/~widelands-dev/widelands/dynamic_tribe_loading_test/view/head:/src/logic/map_objects/tribes/worker_descr.cc#L72

The problem is that I have no idea why, so I'm not comfortable with merging this. Maybe I have uncovered an underlying bug, or I have done something stupid that I'm not seeing.

Does anybody have any idea?
-- 
https://code.launchpad.net/~widelands-dev/widelands/dynamic_tribe_loading/+merge/329198
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/dynamic_tribe_loading.