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

Please improve explanation of positional arguments #371

Open
VictorEijkhout opened this issue Sep 23, 2022 · 3 comments
Open

Please improve explanation of positional arguments #371

VictorEijkhout opened this issue Sep 23, 2022 · 3 comments

Comments

@VictorEijkhout
Copy link

The readme is very inadequate. There is a line

options.parse_positional({"script", "server", "filenames"})

which 1. does not end in a semicolon 2. does not have argc/argv unlike the regular parse call, 3. does not assign its result to anything. My best guess

auto kwresult = options.parse_positional( {"foo"} );

has "incomplete type" according to gcc.

Please clarify.

@jarro2783
Copy link
Owner

The parse_positional function sets options as being positional, it doesn't actually parse anything by itself. I will clarify the explanation and maybe change its name too.

@jarro2783
Copy link
Owner

I've updated the readme, does that help?

@VictorEijkhout
Copy link
Author

Much better, thanks.

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