Skip to content

Interpret single letter argument as short#5251

Merged
dlang-bot merged 1 commit intodlang:masterfrom
deadalnix:getopt
Mar 15, 2017
Merged

Interpret single letter argument as short#5251
dlang-bot merged 1 commit intodlang:masterfrom
deadalnix:getopt

Conversation

@deadalnix
Copy link
Contributor

No description provided.

Copy link
Member

@quickfur quickfur left a comment

Choose a reason for hiding this comment

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

Add a unittest?

@deadalnix deadalnix force-pushed the getopt branch 2 times, most recently from bfa84bd to 10bc20b Compare March 14, 2017 00:53
@deadalnix
Copy link
Contributor Author

Add test for splitAndGet

@deadalnix deadalnix force-pushed the getopt branch 2 times, most recently from c6cabd3 to fa6f23a Compare March 14, 2017 13:30
quickfur
quickfur previously approved these changes Mar 14, 2017
@quickfur
Copy link
Member

Hmph, still not sure how the review system works. I'd like at least another reviewer to look at this before we merge.

@quickfur quickfur dismissed their stale review March 14, 2017 22:30

Let's have another reviewer look at this.

@dlang-bot dlang-bot merged commit 3d23778 into dlang:master Mar 15, 2017
@deadalnix deadalnix deleted the getopt branch March 15, 2017 13:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants