The value of this pointer is of no use unless you're using a debugger (or just to see if two things are identical or not) and it's presence produces noise when comparing the output of two runs for testing. Change this by issuing it only if 'verbose' is set. Signed-off-by: Luc Van Oostenryck <luc.vanoostenryck@xxxxxxxxx> --- linearize.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/linearize.c b/linearize.c index 9c2d48aca..7dccf21d8 100644 --- a/linearize.c +++ b/linearize.c @@ -120,7 +120,7 @@ const char *show_pseudo(pseudo_t pseudo) break; } expr = sym->initializer; - snprintf(buf, 64, "<anon symbol:%p>", sym); + snprintf(buf, 64, "<anon symbol:%p>", verbose ? sym : NULL); if (expr) { switch (expr->type) { case EXPR_VALUE: @@ -328,7 +328,7 @@ const char *show_instruction(struct instruction *insn) buf += sprintf(buf, "%s", show_ident(sym->ident)); break; } - buf += sprintf(buf, "<anon symbol:%p>", sym); + buf += sprintf(buf, "<anon symbol:%p>", verbose ? sym : NULL); break; } -- 2.12.0 -- To unsubscribe from this list: send the line "unsubscribe linux-sparse" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html