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@xxxxxxxxxx> --- misc/e2freefrag.c | 4 +--- 1 files changed, 1 insertions(+), 3 deletions(-) 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; } -- 1.7.4.1 -- To unsubscribe from this list: send the line "unsubscribe linux-ext4" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html