]> diplodocus.org Git - nmh/commitdiff
test/getfullname.c: Move return case first in if-else chain.
authorRalph Corderoy <ralph@inputplus.co.uk>
Sat, 9 Sep 2017 21:08:43 +0000 (22:08 +0100)
committerRalph Corderoy <ralph@inputplus.co.uk>
Sat, 9 Sep 2017 21:08:43 +0000 (22:08 +0100)
test/getfullname.c

index 0e12fe9389fde896a80b53a7cfb6ce0087254a7b..89a5491037e1253ee0018041ae8e26d39ba0c3d2 100644 (file)
@@ -22,9 +22,12 @@ main(int argc, char *argv[])
        struct passwd *pwd;
        char buf[BUFSIZ], *p;
 
+        if (argc > 2) {
+               fprintf (stderr, "usage: %s [name]\n", argv[0]);
+               return 1;
+       }
        if (argc < 2) {
                pwd = getpwuid(getuid());
-
                if (! pwd) {
                        fprintf(stderr, "Unable to retrieve user info for "
                                "userid %ld\n", (long) getuid());
@@ -33,12 +36,8 @@ main(int argc, char *argv[])
 
                strncpy(buf, pwd->pw_gecos, sizeof(buf));
                buf[sizeof(buf) - 1] = '\0';
-       } else if (argc == 2) {
+       } else
                strncpy(buf, argv[1], sizeof(buf));
-       } else if (argc > 2) {
-               fprintf (stderr, "usage: %s [name]\n", argv[0]);
-               return 1;
-       }
 
        /*
         * Perform the same processing that getuserinfo() does.