about summary refs log tree commit diff
diff options
context:
space:
mode:
authorJosh Triplett <josh@joshtriplett.org>2018-09-01 12:24:03 -0700
committerJosh Triplett <josh@joshtriplett.org>2018-09-01 12:24:03 -0700
commit2893a2127d02de058bb41bde44eaa8c8cbd473a4 (patch)
tree0f5b1470820defd843222db3203bd758d145f304
parent0f40a12ea35d0a33cef29dc50787ed6efd74cbfd (diff)
downloadrust-2893a2127d02de058bb41bde44eaa8c8cbd473a4.tar.gz
rust-2893a2127d02de058bb41bde44eaa8c8cbd473a4.zip
tidy: Clean up argument handling
Use `.nth(n)` rather than `.skip(n).next()` (also fixes a clippy
warning), and use `.into()` and a type signature rather than
`PathBuf::from`.
-rw-r--r--src/tools/tidy/src/main.rs7
1 files changed, 2 insertions, 5 deletions
diff --git a/src/tools/tidy/src/main.rs b/src/tools/tidy/src/main.rs
index 4fe77f8b58f..22fa954e5fa 100644
--- a/src/tools/tidy/src/main.rs
+++ b/src/tools/tidy/src/main.rs
@@ -24,11 +24,8 @@ use std::path::PathBuf;
 use std::env;
 
 fn main() {
-    let path = env::args_os().skip(1).next().expect("need path to src");
-    let path = PathBuf::from(path);
-
-    let cargo = env::args_os().skip(2).next().expect("need path to cargo");
-    let cargo = PathBuf::from(cargo);
+    let path: PathBuf = env::args_os().nth(1).expect("need path to src").into();
+    let cargo: PathBuf = env::args_os().nth(2).expect("need path to cargo").into();
 
     let args: Vec<String> = env::args().skip(1).collect();