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

Investigate a new approach to handle the app configuration (CONFIG.dat) #100

Open
ruimalheiro opened this issue May 10, 2019 · 2 comments
Open

Comments

@ruimalheiro
Copy link

Currently the win32 app is responsible to create and use a file called CONFIG.dat.

This is handled by a basic parser that enforces a specific structure: variable:value; but makes the whole application harder to maintain. Can also cause parsing errors if the structure is manually changed (not sure of the side effects).

Possibly two better approaches to try would be:

Note: There are some libs for INI files but I don't think it is necessary as it is that easy via Win32 API.

@indirectlylit
Copy link
Contributor

what kind of configuration is stored?

FYI in Qt there's a cross-platform 'settings' class: https://doc.qt.io/qt-5/qsettings.html

@ruimalheiro
Copy link
Author

I think the current configuration stored in the file are related to how the app should behave. E.g:

RUN_AT_LOGIN
RUN_OPEN_BROWSER
DONT_START
OPEN_BROWSER_OPTION
RUN_OPEN_BROWSER
...

FYI in Qt there's a cross-platform 'settings' class: https://doc.qt.io/qt-5/qsettings.html

That's interesting!

I think one requirement is also being able to edit the configuration without having to start the UI. Would be cool if stored in an human readable format.
The Win32 App could also be a command line tool with an option to lunch in GUI mode. 🤔

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

No branches or pull requests

2 participants