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

Add flag to allow UnrecognizedOptions to be treated as non-options #27

Merged
merged 2 commits into from May 28, 2013

Conversation

Projects
None yet
2 participants
@grum
Contributor

grum commented May 2, 2013

Added a flag to allow accepting unrecognized options as non-option arguments.

We needed this change to provide backwards/forwards compatibility of arguments passed around.

I've tried to stick to your coding scheme as much as I could, I'm quite sure you'll find something to pick on (import order, naming). Feel free to do so.

@pholser

This comment has been minimized.

Show comment
Hide comment
@pholser

pholser May 2, 2013

Very interesting!

Would you also add to src/site/apt/examples.apt? It'd be great to see an illustration there of the scenario that led you to add the feature, and hopefully make it easy for others in similar circumstances to see how to solve the problem.

Thanks for your contribution!

pholser commented on 068f598 May 2, 2013

Very interesting!

Would you also add to src/site/apt/examples.apt? It'd be great to see an illustration there of the scenario that led you to add the feature, and hopefully make it easy for others in similar circumstances to see how to solve the problem.

Thanks for your contribution!

@grum

This comment has been minimized.

Show comment
Hide comment
@grum

grum May 2, 2013

Contributor

Yes no problem, will whip something up :)

Contributor

grum commented May 2, 2013

Yes no problem, will whip something up :)

@grum

This comment has been minimized.

Show comment
Hide comment
@grum

grum May 26, 2013

Contributor

I totally forgot I still had this lurking around, can't believe it was 24days ago :D

Contributor

grum commented May 26, 2013

I totally forgot I still had this lurking around, can't believe it was 24days ago :D

pholser added a commit that referenced this pull request May 28, 2013

Merge pull request #27 from Mojang/allowUnrecognized
Add flag to allow UnrecognizedOptions to be treated as non-options

@pholser pholser merged commit ae72f16 into jopt-simple:master May 28, 2013

pholser added a commit that referenced this pull request May 29, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment