diff options
| author | kennytm <kennytm@gmail.com> | 2018-07-24 09:50:00 +0800 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2018-07-24 09:50:00 +0800 |
| commit | 3efc612a930822ea1b9c6749a3d146235c324a83 (patch) | |
| tree | a558997b7277f80d80d3691d0d05d4982622e824 /src/bootstrap | |
| parent | 378ef99eb3849033fa298ce0953c35a6d5a0b16b (diff) | |
| parent | fe588d894fa392fdf787c18a959b342d38d0c71c (diff) | |
Rollup merge of #52642 - ljedrz:expect_fun_call, r=Mark-Simulacrum
Replace a few expect+format combos with unwrap_or_else+panic That way the error strings won't be created if the related errors are not triggered.
Diffstat (limited to 'src/bootstrap')
| -rw-r--r-- | src/bootstrap/bin/rustc.rs | 4 | ||||
| -rw-r--r-- | src/bootstrap/install.rs | 2 |
2 files changed, 3 insertions, 3 deletions
diff --git a/src/bootstrap/bin/rustc.rs b/src/bootstrap/bin/rustc.rs index e81595a8c62..f2b2f6f1eeb 100644 --- a/src/bootstrap/bin/rustc.rs +++ b/src/bootstrap/bin/rustc.rs @@ -326,7 +326,7 @@ fn main() { let start = Instant::now(); let status = cmd .status() - .expect(&format!("\n\n failed to run {:?}", cmd)); + .unwrap_or_else(|_| panic!("\n\n failed to run {:?}", cmd)); let dur = start.elapsed(); let is_test = args.iter().any(|a| a == "--test"); @@ -346,7 +346,7 @@ fn main() { } } - let code = exec_cmd(&mut cmd).expect(&format!("\n\n failed to run {:?}", cmd)); + let code = exec_cmd(&mut cmd).unwrap_or_else(|_| panic!("\n\n failed to run {:?}", cmd)); std::process::exit(code); } diff --git a/src/bootstrap/install.rs b/src/bootstrap/install.rs index 97fd6f77646..cb28698aa3d 100644 --- a/src/bootstrap/install.rs +++ b/src/bootstrap/install.rs @@ -75,7 +75,7 @@ fn install_sh( let libdir_default = PathBuf::from("lib"); let mandir_default = datadir_default.join("man"); let prefix = builder.config.prefix.as_ref().map_or(prefix_default, |p| { - fs::canonicalize(p).expect(&format!("could not canonicalize {}", p.display())) + fs::canonicalize(p).unwrap_or_else(|_| panic!("could not canonicalize {}", p.display())) }); let sysconfdir = builder.config.sysconfdir.as_ref().unwrap_or(&sysconfdir_default); let datadir = builder.config.datadir.as_ref().unwrap_or(&datadir_default); |
