From 4fbf09ed8c496187e226a6b30933b70503ea7c0b Mon Sep 17 00:00:00 2001
From: Martin von Zweigbergk <martinvonz@google.com>
Date: Sun, 12 Feb 2023 21:34:35 -0800
Subject: [PATCH] revset: remove obsolete silencing of Clippy check

---
 lib/src/revset.rs | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/lib/src/revset.rs b/lib/src/revset.rs
index aaaade163..753f5453e 100644
--- a/lib/src/revset.rs
+++ b/lib/src/revset.rs
@@ -1974,9 +1974,6 @@ pub fn evaluate_expression<'repo>(
                 Ok(Box::new(RevWalkRevset { walk }))
             }
         }
-        // Clippy doesn't seem to understand that we collect the iterator in order to iterate in
-        // reverse
-        #[allow(clippy::needless_collect)]
         RevsetExpression::DagRange { roots, heads } => {
             let root_set = roots.evaluate(repo, workspace_ctx)?;
             let candidate_set = heads.ancestors().evaluate(repo, workspace_ctx)?;