From df9797b7918fdfffcae2681fb636bf621cad2fc7 Mon Sep 17 00:00:00 2001 From: pierwill Date: Fri, 11 Mar 2022 13:51:48 -0600 Subject: [PATCH] Remove redundant slicing of whole ranges in `bootstrap` --- src/bootstrap/setup.rs | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/src/bootstrap/setup.rs b/src/bootstrap/setup.rs index e1235829b3aef..aff2b6c3cbfdb 100644 --- a/src/bootstrap/setup.rs +++ b/src/bootstrap/setup.rs @@ -161,9 +161,9 @@ fn rustup_installed() -> bool { } fn stage_dir_exists(stage_path: &str) -> bool { - match fs::create_dir(&stage_path[..]) { + match fs::create_dir(&stage_path) { Ok(_) => true, - Err(_) => Path::new(&stage_path[..]).exists(), + Err(_) => Path::new(&stage_path).exists(), } } @@ -179,7 +179,7 @@ fn attempt_toolchain_link(stage_path: &str) { return; } - if try_link_toolchain(&stage_path[..]) { + if try_link_toolchain(&stage_path) { println!( "Added `stage1` rustup toolchain; try `cargo +stage1 build` on a separate rust project to run a newly-built toolchain" ); @@ -188,7 +188,7 @@ fn attempt_toolchain_link(stage_path: &str) { println!( "To manually link stage 1 build to `stage1` toolchain, run:\n `rustup toolchain link stage1 {}`", - &stage_path[..] + &stage_path ); } } @@ -222,7 +222,7 @@ fn toolchain_is_linked() -> bool { fn try_link_toolchain(stage_path: &str) -> bool { Command::new("rustup") .stdout(std::process::Stdio::null()) - .args(&["toolchain", "link", "stage1", &stage_path[..]]) + .args(&["toolchain", "link", "stage1", &stage_path]) .output() .map_or(false, |output| output.status.success()) }