verify that the code is correct, maybe try reproducing the original error and verifying it doesn't happen with the patch (but might be difficult in this case).
Details
Details
- Reviewers
elexis Imarok - Commits
- rP19563: Avoid NetClient FSM error when pausing while loading the game.
- Trac Tickets
- #4261
Diff Detail
Diff Detail
- Repository
- rP 0 A.D. Public Repository
- Branch
- 4261
- Lint
Lint OK - Unit
No Unit Test Coverage - Build Status
Buildable 1623 Build 2561: Vulcan Build Jenkins Build 2560: arc lint + arc unit
Event Timeline
Comment Actions
Build is green
Updating workspaces. Build (release)... Build (debug)... Running release tests... Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK! Running debug tests... Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
http://jw:8080/job/phabricator/1126/ for more details.
Comment Actions
Build is green
Updating workspaces. Build (release)... Build (debug)... Running release tests... Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK! Running debug tests... Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
http://jw:8080/job/phabricator/1128/ for more details.
binaries/data/mods/public/gui/session/menu.js | ||
---|---|---|
1041 | Why not just add the g_IsNetworkedActive check here? |
Comment Actions
Build is green
Updating workspaces. Build (release)... Build (debug)... Running release tests... Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK! Running debug tests... Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
http://jw:8080/job/phabricator/1137/ for more details.
Comment Actions
with that and the merged early return, I will commit it.
binaries/data/mods/public/gui/session/menu.js | ||
---|---|---|
1044 | elexis proposed The NetServer only supports pausing after all clients finished loading the game as comment |
Comment Actions
Build is green
Updating workspaces. Build (release)... Build (debug)... Running release tests... Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK! Running debug tests... Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
http://jw:8080/job/phabricator/1145/ for more details.
Comment Actions
Build is green
Updating workspaces. Build (release)... Build (debug)... Running release tests... Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK! Running debug tests... Running cxxtest tests (306 tests)..................................................................................................................................................................................................................................................................................................................OK!
http://jw:8080/job/phabricator/1146/ for more details.