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

refactor program options #12

Open
jmjatlanta opened this issue Jun 17, 2021 · 0 comments
Open

refactor program options #12

jmjatlanta opened this issue Jun 17, 2021 · 0 comments

Comments

@jmjatlanta
Copy link
Owner

jmjatlanta commented Jun 17, 2021

Collecting data from the command line and reconciling that with configuration files are not fun, but are a necessity.

Currently, Komodo uses AppInit and AppInit2 to do this process, but:

  • it mixes command line options with business logic (i.e. check that the value is valid, then go adjust some setting, then move on to the next parameter).
  • it is very large and difficult to maintain

Splitting configuration parsing from the logic that uses the parameters means the logic to parse is placed in one area, and the logic that uses them in another.

That is one of the major goals of boost::program_options. Such work is still cumbersome, but I believe this (or something like it) will ease the burden and straighten out some of the complexity.

@jmjatlanta jmjatlanta changed the title Use boost::program_options refactor program options Jun 18, 2021
jmjatlanta pushed a commit that referenced this issue Nov 2, 2021
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

1 participant