Merge pull request #1766 from pickfire/fix-lints

Use .find() instead of .filter().next()
rust-lang · Apr 15, 2019 · 858f5a4 · 858f5a4
2 parents ed45c1d + 4f56cbd
commit 858f5a4
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/src/cli/rustup_mode.rs b/src/cli/rustup_mode.rs
@@ -1140,8 +1140,7 @@ impl FromStr for CompletionCommand {
     fn from_str(s: &str) -> std::result::Result<Self, Self::Err> {
         match COMPLETIONS
             .iter()
-            .filter(|&(val, _)| val.eq_ignore_ascii_case(s))
-            .next()
+            .find(|&(val, _)| val.eq_ignore_ascii_case(s))
         {
             Some(&(_, cmd)) => Ok(cmd),
             None => {
@@ -1160,7 +1159,7 @@ impl FromStr for CompletionCommand {
 
 impl fmt::Display for CompletionCommand {
     fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
-        match COMPLETIONS.iter().filter(|&(_, cmd)| cmd == self).next() {
+        match COMPLETIONS.iter().find(|&(_, cmd)| cmd == self) {
             Some(&(val, _)) => write!(f, "{}", val),
             None => unreachable!(),
         }