tip(1): Fix various warnings and errors emitted by clang
authorJohn Marino <draco@marino.st>
Wed, 22 Oct 2014 12:28:45 +0000 (14:28 +0200)
committerJohn Marino <draco@marino.st>
Wed, 22 Oct 2014 12:32:58 +0000 (14:32 +0200)
The most serious were void/return mismatches, but clang also complained
about ambiguous brackets and mis-matching printf formats.  It also didn't
like one particular K&R-formatted function, but I'm not sure why not.

usr.bin/tip/libacu/hayes.c
usr.bin/tip/libacu/v3451.c
usr.bin/tip/libacu/v831.c
usr.bin/tip/tip/cmds.c
usr.bin/tip/tip/cu.c
usr.bin/tip/tip/log.c
usr.bin/tip/tip/remote.c
usr.bin/tip/tip/tip.c
usr.bin/tip/tip/value.c

index 365732b..66ccc2e 100644 (file)
@@ -138,8 +138,10 @@ hay_disconnect()
        sleep(1);
        ioctl(FD, TIOCSDTR, 0);
        goodbye();
+       return 0;
 }
 
+void
 hay_abort()
 {
 
@@ -228,7 +230,7 @@ error_rep(c)
                printf("Unknown Modem error: %c (0x%x)", c, c);
        }
        printf("\n\r");
-       return;
+       return 0;
 }
 
 /*
index 3a1a6b1..9b08104 100644 (file)
@@ -123,12 +123,14 @@ v3451_dialer(num, acu)
        return (1);
 }
 
+void
 v3451_disconnect()
 {
 
        close(FD);
 }
 
+void
 v3451_abort()
 {
 
index a14e674..4a2c5c9 100644 (file)
@@ -139,6 +139,7 @@ v831_disconnect()
                 ioctl(FD, TIOCNXCL, 0);
         }
         close(FD);
+        return 0;
 }
 
 v831_abort()
@@ -156,6 +157,7 @@ v831_abort()
         if (FD > 0)
                 ioctl(FD, TIOCCDTR, 0);
         close(FD);
+        return 0;
 }
 
 /*
index d924c24..95de0ec 100644 (file)
@@ -507,11 +507,14 @@ out:
        stop_t = time(0);
        fclose(fd);
        signal(SIGINT, f);
-       if (boolean(value(VERBOSE)))
-               if (boolean(value(RAWFTP)))
+       if (boolean(value(VERBOSE))) {
+               if (boolean(value(RAWFTP))) {
                        prtime(" chars transferred in ", stop_t-start_t);
-               else
+               }
+               else {
                        prtime(" lines transferred in ", stop_t-start_t);
+               }
+       }
        write(fildes[1], (char *)&ccc, 1);
        usetchars ();
 }
@@ -716,7 +719,7 @@ consh(int c)
        char buf[256];
        putchar(c);
        if (prompt("Local command? ", buf, sizeof(buf)))
-               return;
+               return 0;
        tiplink (buf, TL_SIGNAL_TIPOUT | TL_VERBOSE);
        return 0;
 }
index 3f9bb4e..20adaeb 100644 (file)
@@ -105,7 +105,7 @@ cumain(int argc, char *argv[])
         * The "cu" host name is used to define the
         * attributes of the generic dialer.
         */
-       (void)snprintf(sbuf, sizeof(sbuf), "cu%d", BR);
+       (void)snprintf(sbuf, sizeof(sbuf), "cu%ld", BR);
        if ((i = hunt(sbuf)) == 0) {
                printf("all ports busy\n");
                exit(3);
@@ -131,5 +131,6 @@ cumain(int argc, char *argv[])
        }
        if (!HW)
                ttysetup(speed(BR));
+       return 0;
 }
 #endif /* INCLUDE_CU_INTERFACE */
index f1926b1..15b1e6d 100644 (file)
@@ -60,11 +60,14 @@ logent(group, num, acu, message)
                warn("flock");
                return;
        }
-       if ((user = getlogin()) == NULL)
-               if ((pwd = getpwuid(getuid())) == NULL)
+       if ((user = getlogin()) == NULL) {
+               if ((pwd = getpwuid(getuid())) == NULL) {
                        user = "???";
-               else
+               }
+               else {
                        user = pwd->pw_name;
+               }
+       }
        t = time(0);
        timestamp = ctime(&t);
        timestamp[24] = '\0';
index da90763..9cac4d0 100644 (file)
@@ -108,14 +108,16 @@ getremcap(host)
        int   stat;
 
        rempath = getenv("REMOTE");
-       if (rempath != NULL)
-               if (*rempath != '/')
+       if (rempath != NULL) {
+               if (*rempath != '/') {
                        /* we have an entry */
                        cgetset(rempath);
+               }
                else {  /* we have a path */
                        db_array[1] = rempath;
                        db_array[2] = _PATH_REMOTE;
                }
+       }
 
        if ((stat = cgetent(&bp, db_array, host)) < 0) {
                if (DV ||
index 1e2cb22..95c1486 100644 (file)
@@ -145,7 +145,7 @@ main(int argc, char *argv[])
         *      is private, we don't want 'ps' or 'w' to find it).
         */
        if (strlen(system) > sizeof(PNbuf) - 1)
-               errx(1, "phone number too long (max = %d bytes)", sizeof PNbuf - 1);
+               errx(1, "phone number too long (max = %lu bytes)", sizeof PNbuf - 1);
        strncpy(PNbuf, system, sizeof(PNbuf) - 1);
        for (p = system; *p; p++)
                *p = '\0';
@@ -521,8 +521,7 @@ speed(n)
 }
 
 int
-any(c, p)
-       char c, *p;
+any(char c, char *p)
 {
        while (p && *p)
                if (*p++ == c)
index bd36def..dd6f625 100644 (file)
@@ -60,11 +60,14 @@ vinit()
                if (p->v_type&ENVIRON)
                        if ((cp = getenv(p->v_name)))
                                p->v_value = cp;
-               if (p->v_type&IREMOTE)
-                       if (p->v_type&STRING)
+               if (p->v_type&IREMOTE) {
+                       if (p->v_type&STRING) {
                                p->v_value = *(char **) address(p->v_value);
-                       else
+                       }
+                       else {
                                number(p->v_value) = *address(p->v_value);
+                       }
+               }
        }
        /*
         * Read the .tiprc file in the HOME directory