Blizzard / s2client-proto

StarCraft II Client - protocol definitions used to communicate with StarCraft II.
MIT License
3.78k stars 431 forks source link

GameSettings' map name always get overwritten #86

Closed T-Jin closed 6 years ago

T-Jin commented 6 years ago

Starting off with the given examples, passing in map name never seemed to work.

Observations:

  1. There seems to be no public getter to retrieve the map name inside GameSetting of the coordinator.
  2. StartGame() and CreateGame() both takes const string& as param. Use empty string as default argument. Will always overwrite the map name with the passed in value.

With these two combined, I'm in a funny scenario. After passing in a map name through command-line argument, I can't read the map name from the coordinator and use it as param for Start/CreateGame().

T-Jin commented 6 years ago

Examples only work b/c map path are hard-coded everywhere.

Perhaps inside Start/CreateGame(), check for empty map name. If it is empty, use the map name inside GameSettings (as oppose to always overwrite the map name)?

T-Jin commented 6 years ago

Moving the issue to s2client-api repo. closing this one.