commit: actually remove boxing from CommitIteratorExt::ids()

Also simplified lifetime bound a bit.
jj-vcs · Apr 4, 2024 · 06f0ab0 · 06f0ab0
1 parent 32afea1
commit 06f0ab0
Showing 1 changed file with 4 additions and 4 deletions.
diff --git a/lib/src/commit.rs b/lib/src/commit.rs
@@ -164,15 +164,15 @@ impl Commit {
 }
 
 pub trait CommitIteratorExt<'c, I> {
-    fn ids(self) -> impl Iterator<Item = &'c CommitId> + 'c;
+    fn ids(self) -> impl Iterator<Item = &'c CommitId>;
 }
 
 impl<'c, I> CommitIteratorExt<'c, I> for I
 where
-    I: Iterator<Item = &'c Commit> + 'c,
+    I: Iterator<Item = &'c Commit>,
 {
-    fn ids(self) -> impl Iterator<Item = &'c CommitId> + 'c {
-        Box::new(self.map(|commit| commit.id()))
+    fn ids(self) -> impl Iterator<Item = &'c CommitId> {
+        self.map(|commit| commit.id())
     }
 }