style: use .filter_map() instead of .flat_map() where appropriate

jj-vcs · Oct 2, 2024 · 5af6507 · 5af6507
1 parent b8bcc89
commit 5af6507
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/Cargo.toml b/Cargo.toml
@@ -134,6 +134,7 @@ testutils = { path = "lib/testutils" }
 
 [workspace.lints.clippy]
 explicit_iter_loop = "deny"
+flat_map_option = "deny"
 implicit_clone = "deny"
 redundant_else = "deny"
 semicolon_if_nothing_returned = "deny"

diff --git a/lib/src/fileset.rs b/lib/src/fileset.rs
@@ -297,7 +297,7 @@ impl FilesetExpression {
     pub fn explicit_paths(&self) -> impl Iterator<Item = &RepoPath> {
         // pre/post-ordering doesn't matter so long as children are visited from
         // left to right.
-        self.dfs_pre().flat_map(|expr| match expr {
+        self.dfs_pre().filter_map(|expr| match expr {
             FilesetExpression::Pattern(pattern) => pattern.as_path(),
             _ => None,
         })