gprofng/testsuite: eliminate bogus casts
authorJan Beulich <jbeulich@suse.com>
Mon, 19 Dec 2022 08:36:00 +0000 (09:36 +0100)
committerJan Beulich <jbeulich@suse.com>
Mon, 19 Dec 2022 08:36:00 +0000 (09:36 +0100)
Casting pointers to unsigned int is generally problematic and hence
compilers tend to warn about such. While here they're used only in
fprintf(), it still seems better to omit such casts, even if only to
avoid setting bad precedents.

gprofng/testsuite/gprofng.display/synprog/so_syn.c
gprofng/testsuite/gprofng.display/synprog/so_syx.c

index 6fc5aa61163bc12c833db54c6fe303cc926f4fee..d6c9331f9d522a7772f259ca4c9fb6ff586c7b5b 100644 (file)
@@ -32,7 +32,7 @@ so_cputime ()
   /* put a memory leak in here */
   (void) malloc (13);
 
-  fprintf (stderr, "so_burncpu @ 0x%08x\n", (unsigned int) so_burncpu);
+  fprintf (stderr, "so_burncpu @ %p\n", so_burncpu);
   so_burncpu ();
 
   wlog ("end of so_cputime", NULL);
index ae7da6f7414aa407155b568f2f87082240ce6b50..1994a0aca11a1587d99d242a511ffa7c7312241a 100644 (file)
@@ -32,7 +32,7 @@ sx_cputime ()
   /* put a memory leak in here */
   (void) malloc (13);
 
-  fprintf (stderr, "sx_burncpu @ 0x%08x\n", (unsigned int) sx_burncpu);
+  fprintf (stderr, "sx_burncpu @ %p\n", sx_burncpu);
   sx_burncpu ();
   wlog ("end of sx_cputime", NULL);
   return 13;