From edf7d1c30c9b53da5f9cd0fa9597d999f0aa1752 Mon Sep 17 00:00:00 2001 From: Ralf Jung Date: Wed, 31 Jul 2019 13:48:15 +0200 Subject: [PATCH] dedup code a bit --- tests/compiletest.rs | 7 ++++--- 1 file changed, 4 insertions(+), 3 deletions(-) diff --git a/tests/compiletest.rs b/tests/compiletest.rs index f88fb0b6b98..21366c3dbf5 100644 --- a/tests/compiletest.rs +++ b/tests/compiletest.rs @@ -25,8 +25,9 @@ fn rustc_lib_path() -> PathBuf { } fn run_tests(mode: &str, path: &str, target: &str, mut flags: Vec) { - // Some flags we always want. - if rustc_test_suite().is_none() { + let in_rustc_test_suite = rustc_test_suite().is_some(); + // Add some flags we always want. + if !in_rustc_test_suite { // Only `-Dwarnings` on the Miri side to make the rustc toolstate management less painful. // (We often get warnings when e.g. a feature gets stabilized or some lint gets added/improved.) flags.push("-Dwarnings -Dunused".to_owned()); // overwrite the -Aunused in compiletest-rs @@ -40,7 +41,7 @@ fn run_tests(mode: &str, path: &str, target: &str, mut flags: Vec) { let mut config = compiletest::Config::default().tempdir(); config.mode = mode.parse().expect("Invalid mode"); config.rustc_path = miri_path(); - if rustc_test_suite().is_some() { + if in_rustc_test_suite { config.run_lib_path = rustc_lib_path(); config.compile_lib_path = rustc_lib_path(); } -- 2.44.0