user0/rust
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Correctly display merged doctest compilation time

Browse source
This commit is contained in:
Guillaume Gomez 2025-09-25 16:42:45 +02:00
parent bbcbc7818b
commit 9877b26fd1
1 changed files with 8 additions and 4 deletions
Download patch file Download diff file Expand all files Collapse all files

12
src/librustdoc/doctest.rs
View file

@ -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);
}
}