more simplification of returned results

Author: kraktus

src/toolchains.rs

@@ -281,9 +281,7 @@ impl Toolchain {
             .map_err(InstallError::Download)?;
         }
 
-        fs::rename(tmpdir.into_path(), dest).map_err(InstallError::Move)?;
-
-        Ok(())
+        fs::rename(tmpdir.into_path(), dest).map_err(InstallError::Move)
     }
 
     pub(crate) fn remove(&self, dl_params: &DownloadParams) -> Result<(), Error> {
@@ -304,9 +302,7 @@ impl Toolchain {
         );
 
         let dir = dl_params.install_dir.join(rustup_name);
-        fs::remove_dir_all(&dir)?;
-
-        Ok(())
+        fs::remove_dir_all(&dir)
     }
 
     pub(crate) fn run_test(&self, cfg: &Config) -> process::Output {