[PATCH 4/4] gprofng/testsuite: eliminate bogus casts
Jan Beulich
jbeulich@suse.com
Fri Dec 16 08:30:30 GMT 2022
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.
--- a/gprofng/testsuite/gprofng.display/synprog/so_syn.c
+++ b/gprofng/testsuite/gprofng.display/synprog/so_syn.c
@@ -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);
--- a/gprofng/testsuite/gprofng.display/synprog/so_syx.c
+++ b/gprofng/testsuite/gprofng.display/synprog/so_syx.c
@@ -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;
More information about the Binutils
mailing list