Navigation Menu

Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Allow use of an INI-file instead of the Registry #757

Merged
merged 1 commit into from Apr 3, 2020

Conversation

taeber
Copy link
Contributor

@taeber taeber commented Feb 10, 2020

This change adds the -conf switch which, if specified, will use an INI-file instead of the Windows Registry.

This affects #709 and #446.

Testing

I tested this using VS2019 on Windows 10.

  1. Open PowerShell and navigate to the folder containing AppleWin.exe.
  2. Run AppleWin.exe -conf config.ini, where config.ini is a non-existing file.
  3. Close AppleWin.
  4. Note the creation of config.ini. Also note that no values in the Registry should be changed (this is easy to test if you remove the AppleWin group first so you can see that nothing is created).
  5. Run AppleWin.exe normally (without -conf). Change some settings. Notice that the Registry is used and the config.ini is unaffected.

This change adds the `-conf` switch which, if specified, will use an
INI-file instead of the Windows Registry.

This addresses AppleWin#709 and AppleWin#446.
@tomcw
Copy link
Contributor

tomcw commented Feb 10, 2020

@taeber - this PR looks simple and neat, so I don't see any issues in accepting it(*). Before I merge it in, I want to give it some testing. I'll try to find some time this week. Thanks!

(*) MSDN says that GetPrivateProfileString() and WritePrivateProfileString() are intended for Win16 compatiblity... so I wonder for how much longer Microsoft will keep them for?

@tomcw tomcw merged commit 2fa22d2 into AppleWin:master Apr 3, 2020
This was referenced Apr 3, 2020
@tomcw tomcw added this to the 1.29.12 milestone Apr 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants