From 0d0837df6973e00c143a04fe652d74e4df856a2e Mon Sep 17 00:00:00 2001 From: Matthieu Longo Date: Mon, 4 Aug 2025 11:04:13 +0100 Subject: [PATCH] libiberty: disable logging of list content for doubly-linked list tests When the doubly-linked list tests were introduced, the tests were printing the content of the list forward and backward. However, this printing is not needed outside of debugging, and confuses people because the output is not only composed of PASS: lines. This patch disables the printing of the list content by default. If one wants to re-enable it for debugging, he can set the macro DUMP_LIST to 1. libiberty/ChangeLog: * testsuite/test-doubly-linked-list.c: disable debug logging on stdout. --- libiberty/testsuite/test-doubly-linked-list.c | 13 ++++++++++--- 1 file changed, 10 insertions(+), 3 deletions(-) diff --git a/libiberty/testsuite/test-doubly-linked-list.c b/libiberty/testsuite/test-doubly-linked-list.c index 1e1fc637653..93fe19a27ae 100644 --- a/libiberty/testsuite/test-doubly-linked-list.c +++ b/libiberty/testsuite/test-doubly-linked-list.c @@ -155,19 +155,26 @@ bool check(const char *op, bool success = true; bool res; - l_print (wrapper->first); +#define DUMP_LIST 0 + + if (DUMP_LIST) + l_print (wrapper->first); + res = run_test (expect, wrapper, false); printf ("%s: test-linked-list::%s: check forward conformity\n", res ? "PASS": "FAIL", op); success &= res; - l_reverse_print (wrapper->last); + if (DUMP_LIST) + l_reverse_print (wrapper->last); + res = run_test (expect, wrapper, true); printf ("%s: test-linked-list::%s: check backward conformity\n", res ? "PASS": "FAIL", op); success &= res; - printf("\n"); + if (DUMP_LIST) + printf("\n"); return success; } -- 2.47.2