Changeset 3065 for trunk/src/kmk/kmkbuiltin

Timestamp:

Sep 30, 2017, 2:52:35 PM (8 years ago)

Author:

bird

Message:

misc gcc warning fixes

Location:

trunk/src/kmk/kmkbuiltin

Files:

• echo.c (modified) (1 diff)
• err.c (modified) (8 diffs)
• test.c (modified) (2 diffs)

trunk/src/kmk/kmkbuiltin/echo.c

@@ -73 +73 @@
 {
         char *errstr = strerror(errno);
+        ssize_t cchIgn = 0; /* this is to shut up irrelevant warnings on linux. */
 #ifdef _MSC_VER
         int doserrno = _doserrno;
-       char szDosErr[48];
-       sprintf(szDosErr, " (doserrno=%d)", doserrno);
+        char szDosErr[48];
+        sprintf(szDosErr, " (doserrno=%d)", doserrno);
 #endif
-        write(STDERR_FILENO, prog, strlen(prog));
-        write(STDERR_FILENO, ": ", 2);
-        write(STDERR_FILENO, reason, strlen(reason));
-        write(STDERR_FILENO, ": ", 2);
-        write(STDERR_FILENO, errstr, strlen(errstr));
+        cchIgn += write(STDERR_FILENO, prog, strlen(prog));
+        cchIgn += write(STDERR_FILENO, ": ", 2);
+        cchIgn += write(STDERR_FILENO, reason, strlen(reason));
+        cchIgn += write(STDERR_FILENO, ": ", 2);
+        cchIgn += write(STDERR_FILENO, errstr, strlen(errstr));
 #ifdef _MSC_VER
-        write(STDERR_FILENO, szDosErr, strlen(szDosErr));
+        cchIgn += write(STDERR_FILENO, szDosErr, strlen(szDosErr));
 #endif
-        write(STDERR_FILENO, "\n", 1);
+        cchIgn += write(STDERR_FILENO, "\n", 1);
+        (void)cchIgn;
 }
 

trunk/src/kmk/kmkbuiltin/err.c

@@ -55 +55 @@
     char szMsg[4096];
     int cchMsg = snprintf(szMsg, sizeof(szMsg), "%s: ", g_progname);
-    if (cchMsg < sizeof(szMsg) - 1 && cchMsg > 0)
+    if (cchMsg < (int)sizeof(szMsg) - 1 && cchMsg > 0)
     {
         int cchMsg2;
@@ -62 +62 @@
         va_end(args);
 
-        if (   cchMsg < sizeof(szMsg) - 1
+        if (   cchMsg < (int)sizeof(szMsg) - 1
             && cchMsg2 >= 0)
         {
             cchMsg += cchMsg2 = snprintf(&szMsg[cchMsg], sizeof(szMsg) - cchMsg, ": %s\n", strerror(error));
-            if (   cchMsg < sizeof(szMsg) - 1
+            if (   cchMsg < (int)sizeof(szMsg) - 1
                 && cchMsg2 >= 0)
             {
@@ -96 +96 @@
     char szMsg[4096];
     int cchMsg = snprintf(szMsg, sizeof(szMsg), "%s: ", g_progname);
-    if (cchMsg < sizeof(szMsg) - 1 && cchMsg > 0)
+    if (cchMsg < (int)sizeof(szMsg) - 1 && cchMsg > 0)
     {
         int cchMsg2;
@@ -103 +103 @@
         va_end(args);
 
-        if (   cchMsg < sizeof(szMsg) - 1
+        if (   cchMsg < (int)sizeof(szMsg) - 1
             && cchMsg2 >= 0)
         {
@@ -132 +132 @@
     char szMsg[4096];
     int cchMsg = snprintf(szMsg, sizeof(szMsg), "%s: ", g_progname);
-    if (cchMsg < sizeof(szMsg) - 1 && cchMsg > 0)
+    if (cchMsg < (int)sizeof(szMsg) - 1 && cchMsg > 0)
     {
         int cchMsg2;
@@ -139 +139 @@
         va_end(args);
 
-        if (   cchMsg < sizeof(szMsg) - 1
+        if (   cchMsg < (int)sizeof(szMsg) - 1
             && cchMsg2 >= 0)
         {
             cchMsg += cchMsg2 = snprintf(&szMsg[cchMsg], sizeof(szMsg) - cchMsg, ": %s\n", strerror(error));
-            if (   cchMsg < sizeof(szMsg) - 1
+            if (   cchMsg < (int)sizeof(szMsg) - 1
                 && cchMsg2 >= 0)
             {
@@ -169 +169 @@
     char szMsg[4096];
     int cchMsg = snprintf(szMsg, sizeof(szMsg), "%s: ", g_progname);
-    if (cchMsg < sizeof(szMsg) - 1 && cchMsg > 0)
+    if (cchMsg < (int)sizeof(szMsg) - 1 && cchMsg > 0)
     {
         int cchMsg2;
@@ -176 +176 @@
         va_end(args);
 
-        if (   cchMsg < sizeof(szMsg) - 1
+        if (   cchMsg < (int)sizeof(szMsg) - 1
             && cchMsg2 >= 0)
         {

trunk/src/kmk/kmkbuiltin/test.c

@@ -278 +278 @@
 #else /* in kmk */
                         /* let job.c spawn the process, make a job.c style argv_spawn copy. */
-                        char *buf, *cur, **argv_new;
+                        char *cur, **argv_new;
                         size_t sz = 0;
                         int argc_new = 0;
@@ -288 +288 @@
 
                         argv_new = xmalloc((argc_new + 1) * sizeof(char *));
-                        buf = cur = xmalloc(sz);
+                        cur = xmalloc(sz);
                         for (i = 0; i < argc_new; i++) {
                                 size_t len = strlen(argv_spawn[i]) + 1;