From 0e65aeb95d230ef5b49b42eebdc3b294bc71fd7a Mon Sep 17 00:00:00 2001 From: Josh Triplett Date: Sat, 1 Sep 2018 17:41:45 -0700 Subject: [PATCH] tidy: features.rs: collect_lib_features: Simplify Use `if let` to simplify a match, and use `contains_key` instead of `get`. --- src/tools/tidy/src/features.rs | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/src/tools/tidy/src/features.rs b/src/tools/tidy/src/features.rs index 05eeae7d6b01..91e6423a6b07 100644 --- a/src/tools/tidy/src/features.rs +++ b/src/tools/tidy/src/features.rs @@ -259,14 +259,11 @@ pub fn collect_lib_features(base_src_path: &Path) -> Features { map_lib_features(base_src_path, &mut |res, _, _| { - match res { - Ok((name, feature)) => { - if lib_features.get(name).is_some() { - return; - } - lib_features.insert(name.to_owned(), feature); - }, - Err(_) => (), + if let Ok((name, feature)) = res { + if lib_features.contains_key(name) { + return; + } + lib_features.insert(name.to_owned(), feature); } }); lib_features