Skip to content

Commit bd12758

Browse files
Auto merge of #149227 - lolbinarycat:rustdoc-run_test-dyn, r=<try>
rustdoc: use dyn Fn in run_test to avoid monomorphization bloat
2 parents 4b1b6dd + 72ee209 commit bd12758

File tree

2 files changed

+3
-3
lines changed

2 files changed

+3
-3
lines changed

‎src/librustdoc/doctest.rs‎

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -553,7 +553,7 @@ fn run_test(
553553
doctest: RunnableDocTest,
554554
rustdoc_options: &RustdocOptions,
555555
supports_color: bool,
556-
report_unused_externs: impl Fn(UnusedExterns),
556+
report_unused_externs: &dyn Fn(UnusedExterns),
557557
) -> (Duration, Result<(), TestFailure>) {
558558
let langstr = &doctest.langstr;
559559
// Make sure we emit well-formed executable names for our target.
@@ -1142,7 +1142,7 @@ fn doctest_run_fn(
11421142
merged_test_code: None,
11431143
};
11441144
let (_, res) =
1145-
run_test(runnable_test, &rustdoc_options, doctest.supports_color, report_unused_externs);
1145+
run_test(runnable_test, &rustdoc_options, doctest.supports_color, &report_unused_externs);
11461146

11471147
if let Err(err) = res {
11481148
match err {

‎src/librustdoc/doctest/runner.rs‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -210,7 +210,7 @@ std::process::Termination::report(test::test_main(test_args, tests, None))
210210
merged_test_code: Some(code),
211211
};
212212
let (duration, ret) =
213-
run_test(runnable_test, rustdoc_options, self.supports_color, |_: UnusedExterns| {});
213+
run_test(runnable_test, rustdoc_options, self.supports_color, &|_: UnusedExterns| {});
214214
(duration, if let Err(TestFailure::CompileError) = ret { Err(()) } else { Ok(ret.is_ok()) })
215215
}
216216
}

0 commit comments

Comments
 (0)