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

Fix #1: Treat single hyphen as free argument #3

Merged
merged 3 commits into from
Feb 4, 2015

Conversation

flpa
Copy link
Contributor

@flpa flpa commented Feb 4, 2015

This is my approach for solving issue #1.
For some explanations, please see commit 7448d8d.

I also created a .gitignore file to ignore FASLs generated while compiling the files.
Regards,
flpa

flpa added 3 commits February 4, 2015 18:06
Modified MAP-PARSED-OPTIONS accordingly and added a test. After considering to throw an error when encountering a single hyphen I decided to handle it as a free argument because it's how other Unix commands and getopts - as far as I know - handle this situation.
astine added a commit that referenced this pull request Feb 4, 2015
Fix #1: Treat single hyphen as free argument
@astine astine merged commit 501849b into astine:master Feb 4, 2015
@astine
Copy link
Owner

astine commented Feb 4, 2015

This seems like a good solution. Thanks for taking care of this; I haven't had the time look at it that I'd have liked.

@flpa
Copy link
Contributor Author

flpa commented Feb 4, 2015

You're welcome! Thanks for unix-options by the way, I like how simple it is to get going with this library.

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.

2 participants