Merge lp:~widelands-dev/widelands/syncstream_on_option into lp:widelands
Proposed by
SirVer
Status: | Merged | ||||
---|---|---|---|---|---|
Merged at revision: | 7830 | ||||
Proposed branch: | lp:~widelands-dev/widelands/syncstream_on_option | ||||
Merge into: | lp:widelands | ||||
Diff against target: |
336 lines (+73/-97) 6 files modified
src/network/netclient.cc (+1/-3) src/network/nethost.cc (+1/-3) src/ui_fsmenu/options.cc (+6/-30) src/ui_fsmenu/options.h (+2/-5) src/wlapplication.cc (+61/-51) src/wlapplication_messages.cc (+2/-5) |
||||
To merge this branch: | bzr merge lp:~widelands-dev/widelands/syncstream_on_option | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
GunChleoc | code | Approve | |
Tino | Approve | ||
Review via email: mp+286004@code.launchpad.net |
Commit message
- Remove --remove-replays and --remove-
- Adds --write-syncstreams option which defaults to true for now. This will give us more debug information for future desyncs.
Description of the change
Now that I finished this debug aid, I think I have another idea why my last lockstep change was not correct.... Still, this get's rid of two useless options and adds a more useful one (though I'd argue that option should probably never be set to false anyways, so maybe we could just remove it).
To post a comment you must log in.
Does compile and the wss file is written on windows.