Skip to content

Commit

Permalink
Fix circular --config for SDL port.
Browse files Browse the repository at this point in the history
When using the command line options, we had a circular issue:
- using `LoadConfig()` before `ReadOpts()` would mean the `--config` is
ignored.
- using `ReadOpts()` before `LoadConfig()` would mean the command line
options were ignored in favor of the default ones.

If we want to use a custom `vbam.ini`, we need to load it with
`--config`. Now we only consider params order on command line.
  • Loading branch information
denisfa authored and rkitover committed Sep 25, 2019
1 parent 1aa8afb commit 806dd7d
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions src/common/ConfigManager.cpp
Expand Up @@ -898,6 +898,7 @@ int ReadOpts(int argc, char ** argv)
preferences = NULL;
OpenPreferences(optarg);
fclose(f);
LoadConfig();
}
break;
case 'd':
Expand Down
2 changes: 1 addition & 1 deletion src/sdl/SDL.cpp
Expand Up @@ -1538,8 +1538,8 @@ int main(int argc, char** argv)
gb_effects_config.surround = false;
gb_effects_config.enabled = false;

ReadOpts(argc, argv);
LoadConfig(); // Parse command line arguments (overrides ini)
ReadOpts(argc, argv);

inputSetKeymap(PAD_1, KEY_LEFT, ReadPrefHex("Joy0_Left"));
inputSetKeymap(PAD_1, KEY_RIGHT, ReadPrefHex("Joy0_Right"));
Expand Down

0 comments on commit 806dd7d

Please sign in to comment.