Changeset 3064 in kBuild

Timestamp:

Sep 30, 2017 11:38:48 AM (7 years ago)

Author:

bird

Message:

kDepObj: warnings

File:

• trunk/src/kmk/kmkbuiltin/kDepObj.c (modified) (7 diffs)

trunk/src/kmk/kmkbuiltin/kDepObj.c

@@ -53 +53 @@
 # define dprintf(a)             printf a
 # define dump(pb, cb, offBase)  depHexDump(pb,cb,offBase)
+# define WITH_DPRINTF
 #else
 # define dprintf(a)             do {} while (0)
 # define dump(pb, cb, offBase)  do {} while (0)
+# undef  WITH_DPRINTF
 #endif
 
@@ -387 +389 @@
                     if (uSeg == KU16_MAX)
                         return kDepErr(1, "%#07lx - Bad LINNUM32 record\n", (long)((const KU8 *)pHdr - pbFile));
+                    K_NOREF(uGrp);
 
                     if (uLinNumType == KU8_MAX)
                     {
+#ifdef WITH_DPRINTF
                         static const char * const s_apsz[5] =
                         {
                             "source file", "listing file", "source & listing file", "file names table", "path table"
                         };
+#endif
                         KU16 uLine;
                         KU8  uReserved;
@@ -404 +409 @@
                         uLine       = *uData.pu16++;
                         uLinNumType = *uData.pu8++;
-                        uReserved   = *uData.pu8++;
-                        cLinNums    = *uData.pu16++;
-                        uSeg2       = *uData.pu16++;
-                        cbLinNames  = *uData.pu32++;
+                        uReserved   = *uData.pu8++;  K_NOREF(uReserved);
+                        cLinNums    = *uData.pu16++; K_NOREF(cLinNums);
+                        uSeg2       = *uData.pu16++; K_NOREF(uSeg2);
+                        cbLinNames  = *uData.pu32++; K_NOREF(cbLinNames);
 
                         dprintf(("LINNUM32: uGrp=%#x uSeg=%#x uSeg2=%#x uLine=%#x (MBZ) uReserved=%#x\n",
@@ -414 +419 @@
                                  cLinNums, cLinNums, cbLinNames, cbLinNames, uLinNumType,
                                  uLinNumType < K_ELEMENTS(s_apsz) ? s_apsz[uLinNumType] : "??"));
+
                         if (uLine != 0)
                             return kDepErr(1, "%#07lx - Bad LINNUM32 record, line %#x (MBZ)\n", (long)((const KU8 *)pHdr - pbFile), uLine);
@@ -479 +485 @@
                             cbRecLeft -= 4+4+4;
 
-                            iFirstCol = *uData.pu32++;
-                            cCols     = *uData.pu32++;
+                            iFirstCol = *uData.pu32++;  K_NOREF(iFirstCol);
+                            cCols     = *uData.pu32++;  K_NOREF(cCols);
                             cLinFiles = *uData.pu32++;
                             dprintf(("%s table header: cLinFiles=%#" KX32_PRI " (%" KU32_PRI ") iFirstCol=%" KU32_PRI " cCols=%" KU32_PRI"\n",
@@ -1047 +1053 @@
                     else
                     {
-                        fprintf(stderr, "%s: syntax error: The '-%c' option takes a value.\n", chOpt);
+                        fprintf(stderr, "%s: syntax error: The '-%c' option takes a value.\n", argv[0], chOpt);
                         return 2;
                     }
@@ -1131 +1137 @@
                     if (pszIgnoreExt)
                     {
-                        fprintf(stderr, "%s: syntax error: The '-e' option can only be used once!\n");
+                        fprintf(stderr, "%s: syntax error: The '-e' option can only be used once!\n", argv[0]);
                         return 2;
                     }