Use getopt #8

Manually merged
silverwizard merged 1 commits from dj3vande/DoublePlusUnzip:getopt into master 5 years ago

Use getopt to handle dpu's command-line options.
No change in functionality, but this will make it easier to handle other command-line options as they're implemented (see: #5 #6 #7).

Use `getopt` to handle `dpu`'s command-line options. No change in functionality, but this will make it easier to handle other command-line options as they're implemented (see: #5 #6 #7).
silverwizard approved these changes 5 years ago
silverwizard left a comment
Owner

Holy shit, is this just replacing the existing functionality?! But only counting to 10?! Gah!

Holy shit, is this just replacing the existing functionality?! But only counting to 10?! Gah!
Poster

This counts as high as you want it to; the 0-9 option handling is a base 10 decoder.
(Because of how getopt works, you can say -1 -2 -3 and it will do the same thing as either the old or new version does with -123. I'm not sure whether that's a stealth feature, or a bug I don't care about.)

This counts as high as you want it to; the 0-9 option handling is a base 10 decoder. (Because of how getopt works, you can say `-1 -2 -3` and it will do the same thing as either the old or new version does with `-123`. I'm not sure whether that's a stealth feature, or a bug I don't care about.)

I misread, get it now

I hate this

I misread, get it now I hate this
silverwizard closed this pull request 5 years ago

I refuse to allow this to meet the need of "use getopt like a real project", despite meeting the need of "use getopt"

I refuse to allow this to meet the need of "use getopt like a real project", despite meeting the need of "use getopt"
The pull request has been manually merged as a1184bd3fb.
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No Assignees
2 Participants
Notifications
Due Date

No due date set.

Dependencies

No dependencies set.

Reference: silverwizard/DoublePlusUnzip#8
Loading…
There is no content yet.