Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1687100-reveal_fields into lp:widelands

2017-06-07 Thread kaputtnik
Hm.. that's strange, because this works also for the first tutorial, where a headquarter is used. The first campaign "A Place to Call Home" works also nicely with a headquarter. -- https://code.launchpad.net/~widelands-dev/widelands/bug-1687100-reveal_fields/+merge/323721 Your team Widelands D

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/bug-1687100-reveal_fields into lp:widelands

2017-06-07 Thread GunChleoc
The difference is the headquarters in the economy tutorial - seems that it is creating vision no matter what we do. -- https://code.launchpad.net/~widelands-dev/widelands/bug-1687100-reveal_fields/+merge/323721 Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/bu

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/net-boost-asio into lp:widelands

2017-06-07 Thread Notabilis
Thanks for testing and the review. I fixed all your nits except the iterator. A foreach-loop would not work since I need it to get the key of the std::map and also since I am modifying the map while running through it. The temporary key-variable (for it->first) could be probably removed when I w

[Widelands-dev] [Merge] lp:~widelands-dev/widelands/net-boost-asio into lp:widelands

2017-06-07 Thread Notabilis
The proposal to merge lp:~widelands-dev/widelands/net-boost-asio into lp:widelands has been updated. Commit Message changed to: Replaced SDL-Net with Boost.Asio. Added IPv6 support for non-metaserver games. For more details, see: https://code.launchpad.net/~widelands-dev/widelands/net-boost-asi

Re: [Widelands-dev] [Merge] lp:~widelands-dev/widelands/net-boost-asio into lp:widelands

2017-06-07 Thread GunChleoc
Review: Approve Testing on Windows was successful with LAN + internet game. Feel free to merge any time - don't forget to set a commit message on top of this page first though. -- https://code.launchpad.net/~widelands-dev/widelands/net-boost-asio/+merge/324364 Your team Widelands Developers is