From ec7d6654368054e44d079f28022c06175556dde3 Mon Sep 17 00:00:00 2001 From: Matt Corallo Date: Thu, 5 Oct 2023 23:57:08 +0000 Subject: [PATCH 1/1] Make clippy shut up about `PartialOrd` and `Ord` both impl'd Clippy gets mad that we have an implementation of `ParialOrd` and `Ord` separately, even though both are identical. Making `ParitalOrd` call `Ord` makes clippy shut up. --- lightning-invoice/src/lib.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/lightning-invoice/src/lib.rs b/lightning-invoice/src/lib.rs index 9695d790..d953795c 100644 --- a/lightning-invoice/src/lib.rs +++ b/lightning-invoice/src/lib.rs @@ -504,7 +504,7 @@ pub struct Bolt11InvoiceSignature(pub RecoverableSignature); impl PartialOrd for Bolt11InvoiceSignature { fn partial_cmp(&self, other: &Self) -> Option { - self.0.serialize_compact().1.partial_cmp(&other.0.serialize_compact().1) + Some(self.cmp(other)) } } -- 2.30.2