More clear message about total duration of full collections in gctest

* tests/gctest.c [!NO_CLOCK] (check_heap_stats): Print value of
GC_get_full_gc_total_time() on a standalone line (and with the proper
message).

1 files changed, 8 insertions, 9 deletions

diff --git a/tests/gctest.c b/tests/gctest.c
index 3110a111..7b00f834 100644
--- a/tests/gctest.c
+++ b/tests/gctest.c
@@ -2123,16 +2123,15 @@ void check_heap_stats(void)
 #   ifdef THREADS
       (void)GC_unregister_my_thread(); /* just to check it works (for main) */
 #   endif
-#   ifdef NO_CLOCK
-      GC_printf("Completed %u collections\n", (unsigned)GC_get_gc_no());
-#   elif !defined(PARALLEL_MARK)
-      GC_printf("Completed %u collections in %lu ms\n",
-                (unsigned)GC_get_gc_no(), GC_get_full_gc_total_time());
+
+#   ifndef NO_CLOCK
+      GC_printf("Full collections took %lu ms\n", GC_get_full_gc_total_time());
+#   endif
+#   ifdef PARALLEL_MARK
+      GC_printf("Completed %u collections (using %d marker threads)\n",
+                (unsigned)GC_get_gc_no(), GC_get_parallel() + 1);
 #   else
-      GC_printf("Completed %u collections in %lu ms"
-                " (using %d marker threads)\n",
-                (unsigned)GC_get_gc_no(), GC_get_full_gc_total_time(),
-                GC_get_parallel() + 1);
+      GC_printf("Completed %u collections\n", (unsigned)GC_get_gc_no());
 #   endif
     GC_printf("Collector appears to work\n");
 }