From ca38edeb537746d1ff4dc0b1f3e5386c826fa54e Mon Sep 17 00:00:00 2001
From: Chris Mikkelson <cmikk@fsi.io>
Date: Sun, 28 Apr 2024 15:57:27 -0500
Subject: [PATCH] cargo-clippy fixes

Remove redudant Some(foo()?).

Make PartialOrd more canonical.
---
 src/seekable/merge.rs | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/src/seekable/merge.rs b/src/seekable/merge.rs
index 1e182e3..fb9860a 100644
--- a/src/seekable/merge.rs
+++ b/src/seekable/merge.rs
@@ -32,9 +32,7 @@ impl<S: Seekable> PartialEq for MergerEnt<S> {
 // boilerplate.
 impl<S: Seekable> PartialOrd for MergerEnt<S> {
     fn partial_cmp(&self, other: &Self) -> Option<Ordering> {
-        let sent = self.cur_item.borrow();
-        let oent = other.cur_item.borrow();
-        oent.0.partial_cmp(&sent.0)
+	Some(self.cmp(other))
     }
 }
 
@@ -68,7 +66,7 @@ where
         active: BinaryHeap::from(
             sources
                 .into_iter()
-                .filter_map(|i| Some(MergerEnt::new(i)?))
+                .filter_map(|i| MergerEnt::new(i))
                 .collect::<Vec<MergerEnt<S>>>(),
         ),
         ended: Vec::new(),
-- 
2.50.1