Skip to content

Commit cd1228e

Browse files
committed
Remove RunnableDoctest
It should instead be the actual input to the running logic. Currently it's not actually quite runnable since it's still missing some information.
1 parent d6b28fd commit cd1228e

File tree

1 file changed

+17
-24
lines changed

1 file changed

+17
-24
lines changed

src/librustdoc/doctest.rs

Lines changed: 17 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -1087,13 +1087,11 @@ impl CreateRunnableDoctests {
10871087
},
10881088
testfn: test::DynTestFn(Box::new(move || {
10891089
doctest_run_fn(
1090-
RunnableDoctest {
1091-
crate_name,
1092-
rustdoc_test_options,
1093-
opts,
1094-
path,
1095-
scraped_test: test,
1096-
},
1090+
crate_name,
1091+
rustdoc_test_options,
1092+
opts,
1093+
path,
1094+
test,
10971095
rustdoc_options,
10981096
unused_externs,
10991097
)
@@ -1102,36 +1100,31 @@ impl CreateRunnableDoctests {
11021100
}
11031101
}
11041102

1105-
/// A doctest that is ready to run.
1106-
struct RunnableDoctest {
1103+
fn doctest_run_fn(
11071104
crate_name: String,
1108-
rustdoc_test_options: IndividualTestOptions,
1109-
opts: GlobalTestOptions,
1105+
test_opts: IndividualTestOptions,
1106+
global_opts: GlobalTestOptions,
11101107
path: PathBuf,
11111108
scraped_test: ScrapedDoctest,
1112-
}
1113-
1114-
fn doctest_run_fn(
1115-
runnable_test: RunnableDoctest,
11161109
rustdoc_options: Arc<RustdocOptions>,
11171110
unused_externs: Arc<Mutex<Vec<UnusedExterns>>>,
11181111
) -> Result<(), String> {
11191112
let report_unused_externs = |uext| {
11201113
unused_externs.lock().unwrap().push(uext);
11211114
};
1122-
let no_run = runnable_test.scraped_test.no_run(&rustdoc_options);
1123-
let edition = runnable_test.scraped_test.edition(&rustdoc_options);
1115+
let no_run = scraped_test.no_run(&rustdoc_options);
1116+
let edition = scraped_test.edition(&rustdoc_options);
11241117
let res = run_test(
1125-
&runnable_test.scraped_test.text,
1126-
&runnable_test.crate_name,
1127-
runnable_test.scraped_test.line,
1118+
&scraped_test.text,
1119+
&crate_name,
1120+
scraped_test.line,
11281121
&rustdoc_options,
1129-
runnable_test.rustdoc_test_options,
1130-
runnable_test.scraped_test.langstr,
1122+
test_opts,
1123+
scraped_test.langstr,
11311124
no_run,
1132-
&runnable_test.opts,
1125+
&global_opts,
11331126
edition,
1134-
runnable_test.path,
1127+
path,
11351128
report_unused_externs,
11361129
);
11371130

0 commit comments

Comments
 (0)