From bc8321eabffa23293d16d6758034203a8c7bffda Mon Sep 17 00:00:00 2001 From: Herbert Xu Date: Mon, 27 Oct 2014 16:06:51 +0800 Subject: [BUILTIN] Use error instead of warnx for fatal errors in printf This patch replaces uses of warnx where we abort with error since the effect is the same. The exit status however changes from 1 to 2. Non-fatal errors where we continue are unchanged. Signed-off-by: Herbert Xu --- src/bltin/printf.c | 17 +++++------------ 1 file changed, 5 insertions(+), 12 deletions(-) (limited to 'src') diff --git a/src/bltin/printf.c b/src/bltin/printf.c index ebc74ae..4812e40 100644 --- a/src/bltin/printf.c +++ b/src/bltin/printf.c @@ -132,10 +132,8 @@ int printfcmd(int argc, char *argv[]) argv = argptr; format = *argv; - if (!format) { - warnx("usage: printf format [arg ...]"); - goto err; - } + if (!format) + error("usage: printf format [arg ...]"); gargv = ++argv; @@ -190,10 +188,8 @@ pc: fmt += strspn(fmt, SKIP2); ch = *fmt; - if (!ch) { - warnx("missing format character"); - goto err; - } + if (!ch) + error("missing format character"); /* null terminate format string to we can use it as an argument to printf. */ nextch = fmt[1]; @@ -246,8 +242,7 @@ pc: break; } default: - warnx("%s: invalid directive", start); - goto err; + error("%s: invalid directive", start); } *++fmt = nextch; } @@ -255,8 +250,6 @@ pc: out: return rval; -err: - return 1; } -- cgit 1.4.1