diff options
| author | Josh Triplett <josh@joshtriplett.org> | 2018-09-01 12:24:03 -0700 |
|---|---|---|
| committer | Josh Triplett <josh@joshtriplett.org> | 2018-09-01 12:24:03 -0700 |
| commit | 2893a2127d02de058bb41bde44eaa8c8cbd473a4 (patch) | |
| tree | 0f5b1470820defd843222db3203bd758d145f304 | |
| parent | 0f40a12ea35d0a33cef29dc50787ed6efd74cbfd (diff) | |
| download | rust-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.rs | 7 |
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(); |
