]> Zhao Yanbai Git Server - minix.git/commitdiff
lib: fixes for warnings that clang has for libraries.
authorBen Gras <ben@minix3.org>
Tue, 6 Jul 2010 12:08:22 +0000 (12:08 +0000)
committerBen Gras <ben@minix3.org>
Tue, 6 Jul 2010 12:08:22 +0000 (12:08 +0000)
36 files changed:
lib/libc/ansi/misc.c
lib/libc/ansi/signal.c
lib/libc/ip/getnetbyaddr.c
lib/libc/ip/getnetbyname.c
lib/libc/ip/getproto.c
lib/libc/ip/getprotoent.c
lib/libc/ip/getservent.c
lib/libc/ip/getsrvbyname.c
lib/libc/ip/getsrvbyport.c
lib/libc/ip/inet_network.c
lib/libc/ip/res_comp.c
lib/libc/ip/res_mkquery.c
lib/libc/ip/res_query.c
lib/libc/ip/res_send.c
lib/libc/ip/ruserok.c
lib/libc/ip/servxcheck.c
lib/libc/other/_getsysinfo.c
lib/libc/other/_mount.c
lib/libc/other/hypot.c
lib/libc/other/putenv.c
lib/libc/other/putw.c
lib/libc/other/setenv.c
lib/libc/regex/engine.c
lib/libc/regex/regexec.c
lib/libc/stdio/doprnt.c
lib/libc/stdio/mktemp.c
lib/libc/stdio/vsprintf.c
lib/libcurses/cursesio.c
lib/libcurses/update.c
lib/libcurses/wgetch.c
lib/libedit/editline.c
lib/libsys/panic.c
lib/libsys/sef.c
lib/libz/gzlib.c
lib/libz/gzread.c
lib/libz/gzwrite.c

index 2ea0b88a6edfcc4f4e522198e15e16203f42dcc5..3a7139c75ce669ba830dbe858f135fe8b7845b60 100644 (file)
@@ -181,7 +181,7 @@ parseRule(register char *buf, register const char *p)
        if (*p == '/') {
                q = ++p;
                if (!(p = parseTime(&time, p, &dstend))) return NULL;
-               while(*buf++ = *q++);
+               while((*buf++ = *q++));
        }
        if (*p) return NULL;
        return p;
index ff476e0b9791cdae16e23c547899ebb7b590d7fd..f00d38dd6fce245752fd9e311774fc72586a7f79 100644 (file)
@@ -15,7 +15,7 @@ sighandler_t disp;            /* signal handler, or SIG_DFL, or SIG_IGN */
        errno = EINVAL;
        return(SIG_ERR);
   }
-  sigemptyset(&sa.sa_mask);
+  (void) sigemptyset(&sa.sa_mask);
 
 #ifdef WANT_UNRELIABLE_SIGNALS
   /* Allow the signal being handled to interrupt the signal handler. */
index 3ddb1009addb4ed1ac60d11dc9be9d618f4436fb..f77bdb661a844b78d7f806e250be662b27d89ed9 100644 (file)
@@ -47,7 +47,7 @@ getnetbyaddr(net, type)
        register struct netent *p;
 
        setnetent(_net_stayopen);
-       while (p = getnetent())
+       while ((p = getnetent()))
                if (p->n_addrtype == type && p->n_net == net)
                        break;
        if (!_net_stayopen)
index c8e92c6546f93891e419e125314ecc6edb9201e1..09924a2c7c164c27d86a9601ceea5bb15d7bf9d3 100644 (file)
@@ -48,7 +48,7 @@ getnetbyname(name)
        register char **cp;
 
        setnetent(_net_stayopen);
-       while (p = getnetent()) {
+       while ((p = getnetent())) {
                if (strcmp(p->n_name, name) == 0)
                        break;
                for (cp = p->n_aliases; *cp != 0; cp++)
index 34bf8ae06866785f9cf34fc3d27101984bda07cf..9d84b13dce0e6cfbad8261bf9d7343e87c402cb6 100644 (file)
@@ -37,7 +37,7 @@ getprotobynumber(proto)
        register struct protoent *p;
 
        setprotoent(_proto_stayopen);
-       while (p = getprotoent())
+       while ((p = getprotoent()))
                if (p->p_proto == proto)
                        break;
        if (!_proto_stayopen)
index 8c56716883b9d0dc6bb3dc619bbfbfdb1a8ec75d..33b466e80a6888829bb72e1d322e19d42bc0ffe3 100644 (file)
@@ -114,7 +114,7 @@ any(cp, match)
 {
        register char *mp, c;
 
-       while (c = *cp) {
+       while ((c = *cp)) {
                for (mp = match; *mp; mp++)
                        if (*mp == c)
                                return (cp);
index 9e89c3823248ce247f60f83ce7100856f11b8841..a8c3337a73dff95d958bbc39d1b6eed44817ed9b 100644 (file)
@@ -115,7 +115,7 @@ any(cp, match)
 {
        register char *mp, c;
 
-       while (c = *cp) {
+       while ((c = *cp)) {
                for (mp = match; *mp; mp++)
                        if (*mp == c)
                                return (cp);
index 1af2e3457f6ec07a0e331dac3f25860ba8c30c84..242f1aa3379ef428fb539a4fdf87ac782533d70f 100644 (file)
@@ -35,7 +35,7 @@ getservbyname(name, proto)
        register char **cp;
 
        setservent(_serv_stayopen);
-       while (p = getservent()) {
+       while ((p = getservent())) {
                if (strcmp(name, p->s_name) == 0)
                        goto gotname;
                for (cp = p->s_aliases; *cp; cp++)
index 066b3592db8226ebaa5a288d6068c531665ea6de..3606873c542b14c1b5c1b50fe6e9d074720516f5 100644 (file)
@@ -38,7 +38,7 @@ getservbyport(port, proto)
        register struct servent *p;
 
        setservent(_serv_stayopen);
-       while (p = getservent()) {
+       while ((p = getservent())) {
                if (p->s_port != port)
                        continue;
                if (proto == 0 || strcmp(p->s_proto, proto) == 0)
index 92513e3425e0550ec42835046792b41d5a706141..15da001061011887097d61f623b53c95227575e5 100644 (file)
@@ -62,7 +62,7 @@ again:
                base = 8, cp++;
        if (*cp == 'x' || *cp == 'X')
                base = 16, cp++;
-       while (c = *cp) {
+       while ((c = *cp)) {
                if (isdigit(c)) {
                        val = (val * base) + (c - '0');
                        cp++;
index 66b772b7d0f66aa710746e6b9fcab985a84b7c07..44c0129f75ba98b6f11403c8dc5ebce211f04dec 100644 (file)
@@ -58,6 +58,7 @@ static int dn_find();
  * 'exp_dn' is a pointer to a buffer of size 'length' for the result.
  * Return size of compressed name or -1 if there was an error.
  */
+int
 dn_expand(msg, eomorig, comp_dn, exp_dn, length)
        CONST u_char *msg, *eomorig, *comp_dn;
        u_char *exp_dn;
@@ -75,7 +76,7 @@ dn_expand(msg, eomorig, comp_dn, exp_dn, length)
        /*
         * fetch next label in domain name
         */
-       while (n = *cp++) {
+       while ((n = *cp++)) {
                /*
                 * Check for indirection
                 */
@@ -220,6 +221,7 @@ dn_comp(exp_dn, comp_dn, length, dnptrs, lastdnptr)
 /*
  * Skip over a compressed domain name. Return the size or -1.
  */
+int
 dn_skipname(comp_dn, eom)
        CONST u_char *comp_dn, *eom;
 {
@@ -264,7 +266,7 @@ dn_find(exp_dn, msg, dnptrs, lastdnptr)
        for (cpp = dnptrs; cpp < lastdnptr; cpp++) {
                dn = exp_dn;
                sp = cp = *cpp;
-               while (n = *cp++) {
+               while ((n = *cp++)) {
                        /*
                         * check for indirection
                         */
index ef6a0ab0f571f7dec9fecbee0cd5c562fe86af94..132eeb894c6cebf95c06130917dabfa21aa8fada 100644 (file)
@@ -55,6 +55,7 @@ static char sccsid[] = "@(#)res_mkquery.c     6.12 (Berkeley) 6/1/90";
  * Form all types of queries.
  * Returns the size of the result or -1.
  */
+int
 res_mkquery(op, dname, class, type, data, datalen, newrr, buf, buflen)
        int op;                 /* opcode of query */
        _CONST char *dname;     /* domain name */
index 7f8f57456b42584417e2b49267190c2c795ba71e..9bc4ad3a1198129b36227623b9719b4a318d11cd 100644 (file)
@@ -154,6 +154,7 @@ res_query(name, class, type, answer, anslen)
  * Only useful for queries in the same name hierarchy as the local host
  * (not, for example, for host address-to-name lookups in domain in-addr.arpa).
  */
+int
 res_search(name, class, type, answer, anslen)
        char *name;             /* domain name */
        int class, type;        /* class and type of query */
index 7e5c47c174804ac43f92620ebf4b7962931f0a49..980e9de9a7bfe1f7317386cf08738ebab749935d 100644 (file)
@@ -105,6 +105,7 @@ static struct sockaddr no_addr;
 #endif /* FD_SET */
 #endif /* _MINIX */
 
+int
 res_send(buf, buflen, answer, anslen)
        const char *buf;
        int buflen;
index ec00a792f7352ba092ecaeba52918db921541a29..f4d946dcfe4478c5bb5348bfa6d110bd7408f1b3 100644 (file)
@@ -61,18 +61,18 @@ static int match(const char *word, const char *pattern)
 #define lc(c, d) ((((c)= (d)) - 'A') <= ('Z' - 'A') ? (c)+= ('a' - 'A') : 0)
 
     for (;;) {
-       lc(cw, *word);
-       lc(cp, *pattern);
+       (void) lc(cw, *word);
+       (void) lc(cp, *pattern);
 
        if (cp == '*') {
            do pattern++; while (*pattern == '*');
-           lc(cp, *pattern);
+           (void) lc(cp, *pattern);
            if (cp == 0) return 1;
 
            while (cw != 0) {
                if (cw == cp && match(word+1, pattern+1)) return 1;
                word++;
-               lc(cw, *word);
+               (void) lc(cw, *word);
            }
            return 0;
        } else
index db5d4e9f7be9b9cca8696191f1f165fffbfdb3a2..dc10dcd751da95e1ff21e4ca2ebe6c44b7e24ad2 100644 (file)
@@ -96,18 +96,18 @@ static int match(const char *word, const char *pattern)
 #define lc(c, d) ((((c)= (d)) - 'A') <= ('Z' - 'A') ? (c)+= ('a' - 'A') : 0)
 
     for (;;) {
-       lc(cw, *word);
-       lc(cp, *pattern);
+       (void) lc(cw, *word);
+       (void) lc(cp, *pattern);
 
        if (cp == '*') {
            do pattern++; while (*pattern == '*');
-           lc(cp, *pattern);
+           (void) lc(cp, *pattern);
            if (cp == 0) return 1;
 
            while (cw != 0) {
                if (cw == cp && match(word+1, pattern+1)) return 1;
                word++;
-               lc(cw, *word);
+               (void) lc(cw, *word);
            }
            return 0;
        } else
index bd921309171037f1a571762fed8209e253b8fa36..e43b3d4d828151cfde1ece5f699b1d9dfb27969f 100644 (file)
@@ -15,6 +15,7 @@
 #include <minix/type.h>
 #include <minix/const.h>
 #include <stdio.h>
+#include <stdlib.h>
 
 #include <machine/archtypes.h>
 #include "../../../kernel/proc.h"
index f846d6ec73b57666f7fab24d10f5537cc40788ae..952963a5099c3c3a63d45fd009f34bbed060888b 100644 (file)
@@ -65,7 +65,7 @@ int mountflags;
        sprintf(label, "fs_%.12s", p);
   } else {
        if (stat(name, &statbuf) < 0) return -1;
-       sprintf(label, "fs_%04x%x", statbuf.st_dev, statbuf.st_ino);
+       sprintf(label, "fs_%04x%lx", statbuf.st_dev, statbuf.st_ino);
   }
 
   /* Tell VFS that we are passing in a 16-byte label. */
index 8682e885f21059056674d39e07a0278a0edba409..2d9e992338e4243a7b826ea62d56f550cd7d3489 100644 (file)
@@ -12,7 +12,6 @@ struct complex {
 };
 
 _PROTOTYPE(double hypot, (double x, double y ));
-_PROTOTYPE(double cabs, (struct complex p_compl ));
 
 /* $Header$ */
 
@@ -34,9 +33,14 @@ hypot(double x, double y)
        return y*sqrt(x*x+1.0);
 }
 
+#if 0
+
+_PROTOTYPE(double cabs, (struct complex p_compl ));
+
 double
 cabs(p_compl)
 struct complex p_compl;
 {
        return hypot(p_compl.r, p_compl.i);
 }
+#endif
index 455ad5c5b0e5b6f50cdb2c0a1e302af1bc940bf7..1b9cefc571a70fde3e3f03245ef5624a059b77bc 100644 (file)
@@ -26,7 +26,7 @@ char *name;
 
        if (!name) return 0;
        if (*_penviron == NULL) return 1;
-       if (r = strchr(name, '=')) {
+       if ((r = strchr(name, '='))) {
                register _CONST char *p, *q;
 
                *r = '\0';
@@ -65,7 +65,7 @@ char *name;
                size = i;
                p = *_penviron;
                *_penviron = v;
-               while (*v++ = *p++);            /* copy the environment */
+               while ((*v++ = *p++));          /* copy the environment */
                v = *_penviron;
        } else if (!(size % ENTRY_INC)) {
                if (!(v = realloc(*_penviron, rounded(size) * sizeof(char **))))
index 807f67f4e91aea5a7dca5a0a37e3d26931c6543b..a7686e9ce79f312a40863ec46331a4ea58fd4273 100644 (file)
@@ -16,7 +16,7 @@ register FILE *stream;
        register char *p = (char *) &w;
 
        while (cnt--) {
-               putc(*p++, stream);
+               (void) putc(*p++, stream);
        }
        if (ferror(stream)) return EOF;
        return w;
index 2c0dbfce709b6babd0226ad66b6e6f1a8d7b2288..a04076f5f9713af0bd891041cb71dc272a8d6d47 100644 (file)
@@ -47,6 +47,7 @@ char *__findenv(const char *, int *);
  *     Set the value of the environmental variable "name" to be
  *     "value".  If rewrite is set, replace any current value.
  */
+int
 setenv(name, value, rewrite)
        register const char *name;
        register const char *value;
@@ -64,7 +65,7 @@ setenv(name, value, rewrite)
                if (!rewrite)
                        return (0);
                if (strlen(c) >= l_value) {     /* old larger; copy over */
-                       while (*c++ = *value++);
+                       while ((*c++ = *value++));
                        return (0);
                }
        } else {                                        /* create new slot */
@@ -94,7 +95,7 @@ setenv(name, value, rewrite)
            malloc((size_t)((int)(c - name) + l_value + 2))))
                return (-1);
        for (c = environ[offset]; (*c = *name++) && *c != '='; ++c);
-       for (*c++ = '='; *c++ = *value++;);
+       for (*c++ = '='; (*c++ = *value++););
        return (0);
 }
 
index b3e5f5bad50eb8aae16d021358cc1f3863b180b2..14f6c067792d1fb1be566b9b5e8403aed1685dfc 100644 (file)
@@ -686,6 +686,7 @@ sopno lev;                  /* PLUS nesting level */
        /* "can't happen" */
        assert(nope);
        /* NOTREACHED */
+       _exit(1);
 }
 
 /*
index 7890a88c4a6fadd6caeccab447257451fe3fe1f0..fe7335b1425a08e9798dd62998b510cd00d1e266 100644 (file)
@@ -53,6 +53,7 @@ static char sccsid[] = "@(#)regexec.c 8.3 (Berkeley) 3/20/94";
 #include <stdlib.h>
 #include <string.h>
 #include <limits.h>
+#include <unistd.h>
 #ifdef __minix_vmd
 #include <bsd/asciictype.h>
 #else
index 432b5edce1b509562350b7eaa161248eb054e1b1..22e97851f129e01f6199699d8171a387f5b1d213 100644 (file)
@@ -131,7 +131,7 @@ _doprnt(register const char *fmt, va_list ap, FILE *stream)
        const char      *oldfmt;
        char            *s1, buf[1025];
 
-       while (c = *fmt++) {
+       while ((c = *fmt++)) {
                if (c != '%') {
 #ifdef CPM
                        if (c == '\n') {
index fbd0218026ae5cb49acd0341cb9853baec256599..4cd712163137129ef06439c11dcc909a23760830 100644 (file)
@@ -46,6 +46,7 @@ static char sccsid[] = "@(#)mktemp.c  8.1 (Berkeley) 6/4/93";
 
 static int _gettemp(char*,int*);
 
+int
 mkstemp(path)
        char *path;
 {
@@ -61,7 +62,7 @@ mktemp(path)
        return(_gettemp(path, (int *)NULL) ? path : (char *)NULL);
 }
 
-static
+static int
 _gettemp(path, doopen)
        char *path;
        register int *doopen;
index 7452a29e0f328ed4dd3cb5f4b5fb17cf1750a416..7f0cc2f9e784a393601339250309b9fba9f96026 100644 (file)
@@ -22,7 +22,7 @@ vsnprintf(char *s, size_t n, const char *format, va_list arg)
 
        retval = _doprnt(format, arg, &tmp_stream);
        tmp_stream._count  = 1;
-       putc('\0',&tmp_stream);
+       (void) putc('\0',&tmp_stream);
 
        return retval;
 }
index 22e75e40acc216528d0df3469546885390abc837..faf72087960cfd6ea37f84f14ac65a5d5d581766 100644 (file)
@@ -48,7 +48,7 @@ char *s;
 void outc(c)
 int c;
 {
-  putchar(c);
+  (void) putchar(c);
 }
 
 /* Move cursor to r,c */
index 44632608555b27d4d31d0150ac0a054a938d947d..7c027d3996d6b64acb0b14c04d114c0216d83f37 100644 (file)
@@ -58,7 +58,7 @@ int ch;
 {
   if ((_cursvar.cursrow < LINES) || (_cursvar.curscol < COLS)) {
        newattr(ch);
-       putchar(ch);
+       (void) putchar(ch);
   }
 }
 
index 1582f10b29032140ad4240e63193e215d2eaf54e..88768558ae97703023df0e64586667076be538b6 100644 (file)
@@ -17,7 +17,7 @@ WINDOW *win;
   }
   inp = getchar();
   if (_cursvar.echoit) {
-       mvwaddch(curscr, win->_cury + win->_begy,
+       (void) mvwaddch(curscr, win->_cury + win->_begy,
                 win->_curx + win->_begx, inp);
        waddch(win, inp);
   }
index 9af0e698273ac166f9b3fce325e25729052fdba1..355063a19dee2ebc47fafd3a2c84a2ef64efa8b8 100644 (file)
@@ -5,6 +5,8 @@
 #include <signal.h>
 #include <errno.h>
 #include <ctype.h>
+#include <termcap.h>
+#include <unistd.h>
 
 /*
 **  Manifest constants.
index 464101d81478db1f5bc701861f58a391c88fd4df..5103fd7af2f28b6965234429c089626b79298c9e 100644 (file)
@@ -18,12 +18,8 @@ PUBLIC void panic(const char *fmt, ...)
   endpoint_t me = NONE;
   char name[20];
   void (*suicide)(void);
-  static int panicing= 0;
   va_list args;
 
-  if(panicing) return;
-  panicing= 1;
-
   if(sys_whoami(&me, name, sizeof(name)) == OK && me != NONE)
        printf("%s(%d): panic: ", name, me);
   else
index 9857225343d2795521b45128d1afeadc81a34188..212dd9c9e8b3cb10d2fab8e8bbfb1b437725bfc2 100644 (file)
@@ -166,6 +166,7 @@ PUBLIC void _exit(int status)
 {
 /* Make exit() an alias for sef_exit() for system services. */
   sef_exit(status);
+  panic("sef_exit failed");
 }
 
 /*===========================================================================*
@@ -175,6 +176,7 @@ PUBLIC void __exit(int status)
 {
 /* Make exit() an alias for sef_exit() for system services. */
   sef_exit(status);
+  panic("sef_exit failed");
 }
 
 /*===========================================================================*
index 603e60ed5444e0e7a2ed9590a9d9a73ce7cc97bf..19b1dc2ab14a68fd207f71d5cf4dc41158a49fd6 100644 (file)
@@ -3,6 +3,8 @@
  * For conditions of distribution and use, see copyright notice in zlib.h
  */
 
+#include <unistd.h>
+
 #include "gzguts.h"
 
 #if defined(_LARGEFILE64_SOURCE) && _LFS64_LARGEFILE-0
index 548201ab0099b51556da32c4873d1dbb2d211b5a..e33998117e388383fcd5b400a35b0c2eddb97300 100644 (file)
@@ -3,6 +3,8 @@
  * For conditions of distribution and use, see copyright notice in zlib.h
  */
 
+#include <unistd.h>
+
 #include "gzguts.h"
 
 /* Local functions */
@@ -167,12 +169,12 @@ local int gz_head(state)
                 gz_error(state, Z_DATA_ERROR, "unknown header flags set");
                 return -1;
             }
-            NEXT();                 /* modification time */
-            NEXT();
-            NEXT();
-            NEXT();
-            NEXT();                 /* extra flags */
-            NEXT();                 /* operating system */
+            (void) NEXT();                 /* modification time */
+            (void) NEXT();
+            (void) NEXT();
+            (void) NEXT();
+            (void) NEXT();                 /* extra flags */
+            (void) NEXT();                 /* operating system */
             if (flags & 4) {        /* extra field */
                 len = (unsigned)NEXT();
                 len += (unsigned)NEXT() << 8;
@@ -187,8 +189,8 @@ local int gz_head(state)
                 while (NEXT() > 0)
                     ;
             if (flags & 2) {        /* header crc */
-                NEXT();
-                NEXT();
+                (void) NEXT();
+                (void) NEXT();
             }
             /* an unexpected end of file is not checked for here -- it will be
                noticed on the first request for uncompressed data */
index e8defc6887a186055f17597968c4fc5791332811..bcd150f022dea8856f8ae1dfbbe93f90a7d76dcf 100644 (file)
@@ -5,6 +5,8 @@
 
 #include "gzguts.h"
 
+#include <unistd.h>
+
 /* Local functions */
 local int gz_init OF((gz_statep));
 local int gz_comp OF((gz_statep, int));