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

Consider to use a command line argument parsing library #558

Open
IAmNotHanni opened this issue Jul 23, 2024 · 0 comments
Open

Consider to use a command line argument parsing library #558

IAmNotHanni opened this issue Jul 23, 2024 · 0 comments
Labels
cat:refactor refactor/clean up/simplifications/etc. diff:first issue good first issue to start contributing org:discussion needs further discussion with others prio:low This has low priority.

Comments

@IAmNotHanni
Copy link
Member

Is your feature request related to a problem?

Our current command line argument parsing code works, but the problem is that it's still easy to crash. Another downside is that if invalid arguments are specified, our code does not generate an "usage" text which demonstrates which command line arguments are available and how to use them. To save time, we should consider to use something like https://github.com/CLIUtils/CLI11

Description

Replace cla_parser.hpp with https://github.com/CLIUtils/CLI11

Alternatives

Keep cla_parser code

Affected Code

https://github.com/inexorgame/vulkan-renderer/tree/main/src/vulkan-renderer/tools

Operating System

All

Additional Context

None

@IAmNotHanni IAmNotHanni added org:triage labels have to be specified or checked prio:low This has low priority. cat:refactor refactor/clean up/simplifications/etc. org:discussion needs further discussion with others diff:first issue good first issue to start contributing and removed org:triage labels have to be specified or checked labels Jul 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cat:refactor refactor/clean up/simplifications/etc. diff:first issue good first issue to start contributing org:discussion needs further discussion with others prio:low This has low priority.
Projects
None yet
Development

No branches or pull requests

1 participant