diff options
author | Lucas De Marchi <lucas.demarchi@profusion.mobi> | 2012-11-05 02:01:06 -0200 |
---|---|---|
committer | Lucas De Marchi <lucas.demarchi@profusion.mobi> | 2012-11-05 02:01:06 -0200 |
commit | 61c48db360c2147b54b92163152c5e4b931d4ff5 (patch) | |
tree | 45a45cd7987ce485103577825b505edece305979 | |
parent | d9a2e1550453283ba28e6f6a6ea7bb0f72a062d9 (diff) | |
download | kmod-61c48db360c2147b54b92163152c5e4b931d4ff5.tar.gz |
depmod: prefer ERR and WRN over plain fprintf
-rw-r--r-- | tools/depmod.c | 18 |
1 files changed, 7 insertions, 11 deletions
diff --git a/tools/depmod.c b/tools/depmod.c index 670b90a..2dcb399 100644 --- a/tools/depmod.c +++ b/tools/depmod.c @@ -2615,14 +2615,12 @@ static int do_depmod(int argc, char *argv[]) case 'q': case 'r': case 'm': - if (idx > 0) { - fprintf(stderr, - "ignored deprecated option --%s\n", - cmdopts[idx].name); - } else { - fprintf(stderr, - "ignored deprecated option -%c\n", c); - } + if (idx > 0) + WRN("Ignored deprecated option --%s\n", + cmdopts[idx].name); + else + WRN("Ignored deprecated option -%c\n", c); + break; case 'h': help(basename(argv[0])); @@ -2635,9 +2633,7 @@ static int do_depmod(int argc, char *argv[]) case '?': goto cmdline_failed; default: - fprintf(stderr, - "Error: unexpected getopt_long() value '%c'.\n", - c); + ERR("unexpected getopt_long() value '%c'.\n", c); goto cmdline_failed; } } |