From dcf965d65271c76b3cb73e53b91147905f811f34 Mon Sep 17 00:00:00 2001 From: Marijn Schouten Date: Fri, 11 Jul 2025 05:29:05 +0000 Subject: [PATCH] x: use let-else --- src/tools/x/src/main.rs | 9 ++++----- 1 file changed, 4 insertions(+), 5 deletions(-) diff --git a/src/tools/x/src/main.rs b/src/tools/x/src/main.rs index 8c5735014df7..93167141d34d 100644 --- a/src/tools/x/src/main.rs +++ b/src/tools/x/src/main.rs @@ -19,15 +19,14 @@ const PYTHON3: &str = "python3"; fn python() -> &'static str { - let val = match env::var_os("PATH") { - Some(val) => val, - None => return PYTHON, + let Some(path) = env::var_os("PATH") else { + return PYTHON; }; let mut python2 = false; let mut python3 = false; - for dir in env::split_paths(&val) { + for dir in env::split_paths(&path) { // `python` should always take precedence over python2 / python3 if it exists if dir.join(PYTHON).with_extension(EXE_EXTENSION).exists() { return PYTHON; @@ -103,6 +102,7 @@ fn main() { println!("{version}"); return; } + let current = match env::current_dir() { Ok(dir) => dir, Err(err) => { @@ -110,7 +110,6 @@ fn main() { process::exit(1); } }; - for dir in current.ancestors() { let candidate = dir.join("x.py"); if candidate.exists() {