Message ID | 1316206180-6375-11-git-send-email-sandeen@redhat.com |
---|---|
State | Accepted, archived |
Headers | show |
On Fri, Sep 16, 2011 at 03:49:25PM -0500, Eric Sandeen wrote: > There is no need to print out a "bad option" message; getopt > does that for us, and in fact will change "c" to "?" so > it's not even useful. > > Signed-off-by: Eric Sandeen <sandeen@redhat.com> Applied, thanks. - Ted -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html
diff --git a/misc/e2freefrag.c b/misc/e2freefrag.c index e6fd67e..419c6fc 100644 --- a/misc/e2freefrag.c +++ b/misc/e2freefrag.c @@ -282,10 +282,8 @@ int main(int argc, char *argv[]) } chunk_info.chunkbytes *= 1024; break; - default: - fprintf(stderr, "%s: bad option '%c'\n", - progname, c); case 'h': + default: usage(progname); break; }
There is no need to print out a "bad option" message; getopt does that for us, and in fact will change "c" to "?" so it's not even useful. Signed-off-by: Eric Sandeen <sandeen@redhat.com> --- misc/e2freefrag.c | 4 +--- 1 files changed, 1 insertions(+), 3 deletions(-)