dtc: remove unknown option printf, since getopt will print it
ClosedPublic
Actions

Authored by bcran on Feb 13 2020, 4:03 PM.

Details

Reviewers

kevans
manu
emaste
theraven

Commits

rS358183: dtc: remove unknown option printf, since getopt will print it

Summary

Since we don't set opterr to 0, getopt prints a message when it
encounters an unknown/invalid option. We therefore don't need to
print our own message in the default handler.

Test Plan

Before:
% dtc -z
dtc: illegal option -- z
Unknown option ?

After:
% dtc -z
dtc: illegal option -- z

Diff Detail

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 29357
Build 27256: arc lint + arc unit

Event Timeline

bcran created this revision.Feb 13 2020, 4:03 PM

Harbormaster completed remote builds in B29347: Diff 68249.Feb 13 2020, 4:03 PM

change should also go here: https://github.com/davidchisnall/dtc

In D23662#519454, @emaste wrote:

change should also go here: https://github.com/davidchisnall/dtc

Oh I was wondering if there was an upstream!

Oh I was wondering if there was an upstream!

It's a pseudo-upstream - we don't bring it in via vendor/, but @theraven uses the git repo to coordinate development.

I'd be tempted to drop this one since the other message is more informative -- adding David.

Remove the print statement in the default handler.

Harbormaster completed remote builds in B29357: Diff 68264.Feb 13 2020, 6:41 PM

bcran retitled this revision from dtc: unknown getopt option character is in optopt, not ch to dtc: remove unknown option printf, since getopt will print it.Feb 13 2020, 6:41 PM

bcran edited the summary of this revision. (Show Details)

bcran edited the test plan for this revision. (Show Details)

This looks fine, but please submit it as a GitHub PR so that it goes through CI (I can't imagine it failing).