widelands-dev team mailing list archive

[Klaus Halfmann <klaus.halfmann@xxxxxxxx>]

The proposal to merge lp:~widelands-dev/widelands/refactor_gameclient into lp:widelands has been updated.

Description changed to:

* Refactor big switch statement in gameclient.cc
* Refactor ::run() to improve readability
* use send_player_command with Ptr instead of Reference to clarify ownership of PlayerCommand.
* Added more comments

Added some TODOs and empty comments where I did not know the details.

There should be no functional change at all.

For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/refactor_gameclient/+merge/366743
-- 
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/refactor_gameclient into lp:widelands.