Merge pull request #531 from gemmaro/master

Minor change: Use "args" where there is more than one "arg"
sagiegurari · Mar 10, 2021 · ccce394 · ccce394
2 parents 9c6a676 + 9588640
commit ccce394
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 13 deletions.
diff --git a/src/lib/command.rs b/src/lib/command.rs
@@ -161,9 +161,7 @@ pub(crate) fn run_command_get_output(
 
     match *args {
         Some(ref args_vec) => {
-            for arg in args_vec.iter() {
-                command.arg(arg);
-            }
+            command.args(args_vec);
         }
         None => debug!("No command args defined."),
     };

diff --git a/src/lib/installer/cargo_plugin_installer.rs b/src/lib/installer/cargo_plugin_installer.rs
@@ -20,11 +20,7 @@ fn is_crate_installed(toolchain: &Option<String>, crate_name: &str) -> bool {
         Some(ref toolchain_string) => {
             let command_spec = wrap_command(toolchain_string, "cargo", &None);
             let mut cmd = Command::new(command_spec.command);
-
-            let args_vec = command_spec.args.unwrap();
-            for arg in args_vec.iter() {
-                cmd.arg(arg);
-            }
+            cmd.args(command_spec.args.unwrap());
 
             cmd
         }

diff --git a/src/lib/installer/rustup_component_installer.rs b/src/lib/installer/rustup_component_installer.rs
@@ -17,11 +17,7 @@ pub(crate) fn is_installed(toolchain: &Option<String>, binary: &str, test_args:
         Some(ref toolchain_string) => {
             let command_spec = wrap_command(toolchain_string, binary, &None);
             let mut cmd = Command::new(command_spec.command);
-
-            let args_vec = command_spec.args.unwrap();
-            for arg in args_vec.iter() {
-                cmd.arg(arg);
-            }
+            cmd.args(command_spec.args.unwrap());
 
             cmd
         }