sbase

commit 683d108387cb8e7ac7fdaa03fab77b41a8ba5d44
parent fbda47b96470c2c9278cf4902fb517a2882137b4
Author: FRIGN <dev@frign.de>
Date:   Tue, 17 Mar 2015 00:44:18 +0100

Audit logname(1)

1) Add usage().
2) Idiomatic argv0-setter. We don't use arg.h, as we do not process
   flags or arguments.
3) Remove program-name from eprintf-call. This is done in the eprintf-
   function itself when the DEBUG-define is set.
   We'll activate it by default later.
4) Add empty line before return.

Diffstat:
M
README
 | 
2
+-
M
logname.c
 | 
17
+++++++++++++----

2 files changed, 14 insertions(+), 5 deletions(-)
diff --git a/README b/README
@@ -41,7 +41,7 @@ The following tools are implemented ('*' == finished, '#' == UTF-8 support,
 =*| link            yes                          none
 =*| ln              yes                          none
 =*| logger          yes                          none
-=*  logname         yes                          none
+=*| logname         yes                          none
 =   ls              no                           (-C), -S, -f, -m, -s, -x
 =*| md5sum          non-posix                    none
 =*| mkdir           yes                          none
diff --git a/logname.c b/logname.c
@@ -4,17 +4,26 @@
 
 #include "util.h"
 
+static void
+usage(void)
+{
+	eprintf("usage: %s\n", argv0);
+}
+
 int
 main(int argc, char *argv[])
 {
 	char *login;
 
-	argv0 = argv[0];
-	if (argc != 1)
-		eprintf("usage: %s\n", argv0);
+	argv0 = argv[0], argc--, argv++;
+
+	if (argc)
+		usage();
+
 	if ((login = getlogin()))
 		puts(login);
 	else
-		eprintf("%s: no login name\n", argv0);
+		eprintf("no login name\n");
+
 	return 0;
 }