In D4770#204068, @phosit wrote:

Yes it works as you described.
Some things i would change:

• Single-player > Matches > Load Game directly starts a game. Multiplayer > Host Game > Load Game does not. This should be consistent.

I did that to give the players (in multiplayer mode) the possibility to edit the settings a bit before launching the saved game, to make sure that everyone has the correct civ. I don't really think that it's useful for single player though, as you will always fall back on the civ that you were playing. I could very well remove the load game button for the single player setup menu (I'll bring that up again in one of the following points)

Oh okay! Did you manage to make it work with a clean save?

@phosit mh weird... I have been using this patch both from a windows desktop and an unbutu desktop, and played with a friend using it... What configuration do you use? What crash message do you obtain? I'll try to replicate your crash. I also wondered if this could be due to the fact that the patch is based on a slightly older version of the project (a month ago or so), I'll try to apply it with a more recent version of the project

Don't skip the size test for NMT_SAVED_GAME_START anymore. Instead, define a header of 4 bytes (instead of 3) for that specific message. 1 byte for the message type, and 3 for its size

Fix the lint inlines

Fix the last comment made in MapBrowserControls.js

Update the code according to the review, cleaner syntax and avoid duplicates

Fix a typo in NetMessage.cpp