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

improve options parsing #1616

Merged
merged 9 commits into from
Sep 14, 2024
Merged

Conversation

snoyer
Copy link
Contributor

@snoyer snoyer commented Sep 11, 2024

First pass on the new options IO. Here we improve parsing for the existing bool, int, double, ratio_t types.

I also split the IO tests from the options structure tests (the options tests could possibly be reduced in the future as they currently also do some parsing/formatting test).

Copy link
Contributor

@mwestphal mwestphal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

small comments about doc, maybe the doc should only be in options.h

@mwestphal mwestphal requested a review from Meakk September 14, 2024 11:45
@mwestphal mwestphal merged commit 743f4f7 into f3d-app:master Sep 14, 2024
39 checks passed
@snoyer snoyer deleted the improve-options-io-1 branch September 15, 2024 03:15
Nokse22 pushed a commit to Nokse22/f3d that referenced this pull request Sep 21, 2024
First pass on the new options IO. Here we improve parsing for the existing `bool`, `int`, `double`, `ratio_t` types.

Also split the IO tests from the options structure tests (the options tests could possibly be reduced in the future as they currently also do some parsing/formatting test).
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.

3 participants