widelands

Code review comment for lp:~widelands-dev/widelands/dynamic_tribe_loading

  1. dynamic_tribe_loading
  2. Merge into trunk
Revision history for this message
GunChleoc (gunchleoc) wrote :

This should be ready now.

I added the remaining comments as NOCOM to the code for easier review and addressed all the rest. I have answered in the code.

For full testing, take the datadir from

https://code.launchpad.net/~widelands-dev/widelands/dynamic_tribe_loading_datadir

And start Empire Scenario 4.

review: Needs Resubmitting
Reply

« Back to merge proposal