]> granicus.if.org Git - procps-ng/commitdiff
miscellaneous: silence a whole bunch of clang warnings
authorJim Warner <james.warner@comcast.net>
Mon, 7 Sep 2015 05:00:00 +0000 (00:00 -0500)
committerCraig Small <csmall@enc.com.au>
Mon, 21 Sep 2015 12:34:27 +0000 (22:34 +1000)
[ but most definitely not all of them by a long shot ]

Reference(s):
proc/diskstat.c:186:17: warning: unused variable 'is_disk' [-Wunused-variable]
    int retval, is_disk;
                ^
proc/namespace.c:110:1: warning: control may reach end of non-void function [-Wreturn-type]
}
^
proc/readproc.c:1131:50: warning: address of array 'ent->d_name' will always evaluate to 'true' [-Wpointer-bo
    if(unlikely(unlikely(!ent) || unlikely(!ent->d_name))) return 0;
                                           ~~~~~~^~~~~~
proc/readproc.c:1158:50: warning: address of array 'ent->d_name' will always evaluate to 'true' [-Wpointer-bo
    if(unlikely(unlikely(!ent) || unlikely(!ent->d_name))) return 0;
                                           ~~~~~~^~~~~~
proc/sysinfo.c:45:12: warning: unused variable 'stat_fd' [-Wunused-variable]
static int stat_fd = -1;
           ^
proc/sysinfo.c:49:12: warning: unused variable 'meminfo_fd' [-Wunused-variable]
static int meminfo_fd = -1;
           ^
proc/sysinfo.c:51:12: warning: unused variable 'vminfo_fd' [-Wunused-variable]
static int vminfo_fd = -1;
           ^
proc/sysinfo.c:53:12: warning: unused variable 'vm_min_free_fd' [-Wunused-variable]
static int vm_min_free_fd = -1;
           ^
proc/uptime.c:157:12: warning: unused variable 'realseconds' [-Wunused-variable]
    time_t realseconds;
           ^
proc/uptime.c:158:16: warning: unused variable 'realtime' [-Wunused-variable]
    struct tm *realtime;
               ^
vmstat.c:574:20: warning: format specifies type 'unsigned int' but the argument has type 'unsigned long' [-Wformat]
                   DSTAT(PROCPS_DISKSTAT_READ_TIME),
                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
vmstat.c:578:20: warning: format specifies type 'unsigned int' but the argument has type 'unsigned long' [-Wformat]
                   DSTAT(PROCPS_DISKSTAT_WRITE_TIME),
                   ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
watch.c:230:7: warning: variable 'endptr' is uninitialized when used here [-Wuninitialized]
        if (*endptr == '\0') set_ansi_attribute(0); /* [m treated as [0m */
             ^~~~~~

Signed-off-by: Jim Warner <james.warner@comcast.net>
proc/diskstat.c
proc/namespace.c
proc/readproc.c
proc/sysinfo.c
proc/uptime.c
vmstat.c
watch.c

index 1a7d50c1977eba21103f8e3db506467bbc3425c9..2b8409b04e6a6b6893618f66fba1046859abe7ba 100644 (file)
@@ -183,7 +183,7 @@ PROCPS_EXPORT int procps_diskstat_new (
 PROCPS_EXPORT int procps_diskstat_read (
         struct procps_diskstat *info)
 {
-    int retval, is_disk;
+    int retval;
     char buf[DISKSTAT_LINE_LEN];
     char devname[DISKSTAT_NAME_LEN];
     struct procps_diskstat_dev *disk;
index d61b1b6ff56efad6b44b38d18c2ec8cbf4a02574..cd4f47d571f2bba3c1e41cad347bbbee88bf2fe4 100644 (file)
@@ -107,4 +107,5 @@ PROCPS_EXPORT int procps_ns_read_pid(
         else
             nsp->ns[i] = 0;
     }
+    return 0;
 }
index 9036f11c65b72600859212a8862e653e5c98d6fd..0d2eb7cb30e3122a45716b9be335741697ad05a3 100644 (file)
@@ -1128,7 +1128,7 @@ static int simple_nextpid(PROCTAB *restrict const PT, proc_t *restrict const p)
   char *restrict const path = PT->path;
   for (;;) {
     ent = readdir(PT->procfs);
-    if(unlikely(unlikely(!ent) || unlikely(!ent->d_name))) return 0;
+    if(unlikely(unlikely(!ent) || unlikely(!ent->d_name[0]))) return 0;
     if(likely(likely(*ent->d_name > '0') && likely(*ent->d_name <= '9'))) break;
   }
   p->tgid = strtoul(ent->d_name, NULL, 10);
@@ -1155,7 +1155,7 @@ static int simple_nexttid(PROCTAB *restrict const PT, const proc_t *restrict con
   }
   for (;;) {
     ent = readdir(PT->taskdir);
-    if(unlikely(unlikely(!ent) || unlikely(!ent->d_name))) return 0;
+    if(unlikely(unlikely(!ent) || unlikely(!ent->d_name[0]))) return 0;
     if(likely(likely(*ent->d_name > '0') && likely(*ent->d_name <= '9'))) break;
   }
   t->tid = strtoul(ent->d_name, NULL, 10);
index c9dc9f3cb87b12a3132c0ae586f335a236e7df22..f58d139c50de85d5f84a1569b6625f8792bb2e22 100644 (file)
 "      proc   /proc   proc    defaults\n"                      \
 "  In the meantime, run \"mount proc /proc -t proc\"\n"
 
-#define STAT_FILE    "/proc/stat"
-static int stat_fd = -1;
 #define LOADAVG_FILE "/proc/loadavg"
 static int loadavg_fd = -1;
-#define MEMINFO_FILE "/proc/meminfo"
-static int meminfo_fd = -1;
-#define VMINFO_FILE "/proc/vmstat"
-static int vminfo_fd = -1;
-#define VM_MIN_FREE_FILE "/proc/sys/vm/min_free_kbytes"
-static int vm_min_free_fd = -1;
 
 // As of 2.6.24 /proc/meminfo seems to need 888 on 64-bit,
 // and would need 1258 if the obsolete fields were there.
index e64120522bba85ea46055e197419980508325c19..c692fd0e3f643c4635bd4aa571f7f85e579616df 100644 (file)
@@ -154,8 +154,6 @@ PROCPS_EXPORT char *procps_uptime_sprint_short(void)
     int updecades, upyears, upweeks, updays, uphours, upminutes;
     int pos = 3;
     int comma = 0;
-    time_t realseconds;
-    struct tm *realtime;
     double uptime_secs, idle_secs;
 
     shortbuf[0] = '\0';
index 81579449c47773a5db2b4e97a2d8343bccd495ec..f53eaac940378c7319c6392358c8998de7ccac2a 100644 (file)
--- a/vmstat.c
+++ b/vmstat.c
@@ -542,7 +542,7 @@ static void diskformat(void)
     struct tm *tm_ptr;
     char timebuf[32];
     const char format[] = "%-5s %6lu %6lu %7lu %7lu %6lu %6lu %7lu %7lu %6lu %6lu";
-    const char wide_format[] = "%-5s %9lu %9lu %lu %1u %9lu %9lu %lu %1u %7lu %7lu";
+    const char wide_format[] = "%-5s %9lu %9lu %lu %lu %9lu %9lu %lu %lu %7lu %7lu";
 
     if (procps_diskstat_new(&disk_stat) < 0)
         xerr(EXIT_FAILURE,
diff --git a/watch.c b/watch.c
index 40beacad487767d630aeef8ff56dcc3fe5721f8d..b7dd4412df499b54f01841beefab184a49e6f73a 100644 (file)
--- a/watch.c
+++ b/watch.c
@@ -198,7 +198,7 @@ static void process_ansi(FILE * fp)
 {
        int i, c;
        char buf[MAX_ANSIBUF];
-       char *numstart, *endptr;
+       char *numstart, *endptr = buf;
 
        c = getc(fp);
        if (c != '[') {