[Valgrind-developers] valgrind: r8723 - in trunk: auxprogs callgrind coregrind

[<sv...@va...>]

Author: sewardj
Date: 2008-11-01 23:55:32 +0000 (Sat, 01 Nov 2008)
New Revision: 8723

Log:
Fixes for compilation warnings from the apparently very strict
gcc-4.3.2 shipped with Ubuntu 8.10.



Modified:
   trunk/auxprogs/valgrind-listener.c
   trunk/callgrind/clo.c
   trunk/callgrind/debug.c
   trunk/callgrind/events.c
   trunk/coregrind/m_main.c


Modified: trunk/auxprogs/valgrind-listener.c
===================================================================
--- trunk/auxprogs/valgrind-listener.c	2008-10-30 13:26:29 UTC (rev 8722)
+++ trunk/auxprogs/valgrind-listener.c	2008-11-01 23:55:32 UTC (rev 8723)
@@ -128,7 +128,8 @@
       if (buf[i] == '\n') {
          fprintf(stdout, "\n(%d) ", conn_count);
       } else {
-         fwrite(&buf[i], 1, 1, stdout);
+         __attribute__((unused)) size_t ignored 
+            = fwrite(&buf[i], 1, 1, stdout);
       }
    }
    fflush(stdout);

Modified: trunk/callgrind/clo.c
===================================================================
--- trunk/callgrind/clo.c	2008-10-30 13:26:29 UTC (rev 8722)
+++ trunk/callgrind/clo.c	2008-11-01 23:55:32 UTC (rev 8723)
@@ -279,7 +279,7 @@
     char sp[] = "                                        ";
 
     if (s>40) s=40;
-    VG_(printf)(sp+40-s);
+    VG_(printf)("%s", sp+40-s);
     VG_(printf)("'%s'/%d\n", node->name, node->length);
   }
   for(i=0;i<NODE_DEGREE;i++) {

Modified: trunk/callgrind/debug.c
===================================================================
--- trunk/callgrind/debug.c	2008-10-30 13:26:29 UTC (rev 8722)
+++ trunk/callgrind/debug.c	2008-11-01 23:55:32 UTC (rev 8723)
@@ -41,7 +41,7 @@
     /* max of 40 spaces */
     char sp[] = "                                        ";
     if (s>40) s=40;
-    VG_(printf)(sp+40-s);
+    VG_(printf)("%s", sp+40-s);
 }
 
 void CLG_(print_bb)(int s, BB* bb)
@@ -168,7 +168,7 @@
 	if (es->e[i-1].nextTop == i)
 	  VG_(printf)("| ");
       }
-      VG_(printf)(es->e[i].type->name);
+      VG_(printf)("%s", es->e[i].type->name);
     }
   }
   VG_(printf)("\n");

Modified: trunk/callgrind/events.c
===================================================================
--- trunk/callgrind/events.c	2008-10-30 13:26:29 UTC (rev 8722)
+++ trunk/callgrind/events.c	2008-11-01 23:55:32 UTC (rev 8723)
@@ -180,7 +180,7 @@
 
   for(i=0; i< es->size; i++) {
     if (pos>0) buf[pos++] = ' ';
-    pos += VG_(sprintf)(buf + pos, es->e[i].type->name);
+    pos += VG_(sprintf)(buf + pos, "%s", es->e[i].type->name);
   }
   buf[pos] = 0;
 
@@ -538,7 +538,7 @@
 
   for(i=0; i< em->size; i++) {
     if (pos>0) buf[pos++] = ' ';
-    pos += VG_(sprintf)(buf + pos, em->set->e[em->index[i]].type->name);
+    pos += VG_(sprintf)(buf + pos, "%s", em->set->e[em->index[i]].type->name);
   }
   buf[pos] = 0;
 

Modified: trunk/coregrind/m_main.c
===================================================================
--- trunk/coregrind/m_main.c	2008-10-30 13:26:29 UTC (rev 8722)
+++ trunk/coregrind/m_main.c	2008-11-01 23:55:32 UTC (rev 8723)
@@ -231,7 +231,7 @@
 	 VG_(printf)("    (none)\n");
    }
    if (debug_help) {
-      VG_(printf)(usage2);
+      VG_(printf)("%s", usage2);
 
       if (VG_(details).name) {
          VG_(printf)("  debugging options for %s:\n", VG_(details).name);