widelands-dev team mailing list archive
-
widelands-dev team
-
Mailing list archive
-
Message #10377
Re: [Merge] lp:~widelands-dev/widelands/net-boost-asio into lp:widelands
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 write code like "it++->first". But I at least prefer the better readable version with the variable.
I will give Bunnybot another chance to run, then this will go in.
--
https://code.launchpad.net/~widelands-dev/widelands/net-boost-asio/+merge/324364
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/net-boost-asio.
References