From 2893a2127d02de058bb41bde44eaa8c8cbd473a4 Mon Sep 17 00:00:00 2001 From: Josh Triplett Date: Sat, 1 Sep 2018 12:24:03 -0700 Subject: [PATCH] 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`. --- src/tools/tidy/src/main.rs | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/src/tools/tidy/src/main.rs b/src/tools/tidy/src/main.rs index 4fe77f8b58fe..22fa954e5fa5 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 = env::args().skip(1).collect();