From 9877b26fd14d32064b1f944dd1e79c9c8faccf16 Mon Sep 17 00:00:00 2001 From: Guillaume Gomez Date: Thu, 25 Sep 2025 16:42:45 +0200 Subject: [PATCH] Correctly display merged doctest compilation time --- src/librustdoc/doctest.rs | 12 ++++++++---- 1 file changed, 8 insertions(+), 4 deletions(-) diff --git a/src/librustdoc/doctest.rs b/src/librustdoc/doctest.rs index 95bd31729de1..9499258f983a 100644 --- a/src/librustdoc/doctest.rs +++ b/src/librustdoc/doctest.rs @@ -404,11 +404,15 @@ pub(crate) fn run_tests( std::mem::drop(temp_dir.take()); times.display_times(); }); - } - if nb_errors != 0 { - // We ensure temp dir destructor is called. - std::mem::drop(temp_dir); + } else { + // If the first condition branch exited successfully, `test_main_with_exit_callback` will + // not exit the process. So to prevent displaying the times twice, we put it behind an + // `else` condition. times.display_times(); + } + // We ensure temp dir destructor is called. + std::mem::drop(temp_dir); + if nb_errors != 0 { std::process::exit(test::ERROR_EXIT_CODE); } }