Add minor refactorings #754

tranzystorekk · 2021-09-03T17:51:19Z

Standards checklist:

The PR title is descriptive.
The code compiles (cargo build)
The code passes rustfmt (cargo fmt)
The code passes clippy (cargo clippy)
The code passes tests (cargo test)
Optional: I have tested the code myself
- I also tested that Topgrade skips the step where needed

Highlights:

r-darwish · 2021-09-04T17:59:38Z

src/main.rs

-                _ => {
-                    continue;
-                }
+                Ok(Key::Char('s' | 'S')) => run_shell(),


This is nice. Could you tell me which feature of Rust are you using here?

Nested "or patterns", introduced in Rust 1.53

r-darwish · 2021-09-04T18:01:25Z

This is really nice. Thank you!

Add minor refactorings

c991077

r-darwish reviewed Sep 4, 2021

View reviewed changes

r-darwish merged commit 1478d07 into r-darwish:develop Sep 4, 2021

tranzystorekk deleted the refactor-v1 branch September 4, 2021 18:02