Fix CI cause by rust updating to 1.73 (#269)

`clippy::incorrect_partial_ord_impl_on_ord_type` was failing.

In a few cases this was fixed here, but others were coming from our use
of the `derivative` crate causing clippy to think both were
non-canonical implementations.

Issue here for derivative: mcarton/rust-derivative#115

This was blanket ignored in cml-chain+cml-core to work around those.

chain/rust/src/lib.rs

@@ -1,3 +1,15 @@
+// This recently introduced lint does not play well with the derivative crate.
+// We have both Ord and PartialOrd derive automatically by derivative's proc macros
+// but clippy sees these as hand implementations.
+// Putting this allow locally where it's found did not seem to supress it,
+// likely due to the structure of how the proc macro derives the code.
+// Doing what is suggested by this lint would just result in us actually doing
+// hand implementations of the PartialOrd (an maybe PartialEq) when there's no need,
+// possibly impacting PartialOrd performance on top of being unnecessary and occuring in generated code.
+// Possibly the derivative crate could get updated to suppress this lint
+// from within their proc macros itself. Issue: https://github.com/mcarton/rust-derivative/issues/115
+#![allow(clippy::incorrect_partial_ord_impl_on_ord_type)]
+
 pub mod address;
 pub mod assets;
 pub mod auxdata;

core/rust/src/lib.rs

@@ -1,3 +1,15 @@
+// This recently introduced lint does not play well with the derivative crate.
+// We have both Ord and PartialOrd derive automatically by derivative's proc macros
+// but clippy sees these as hand implementations.
+// Putting this allow locally where it's found did not seem to supress it,
+// likely due to the structure of how the proc macro derives the code.
+// Doing what is suggested by this lint would just result in us actually doing
+// hand implementations of the PartialOrd (an maybe PartialEq) when there's no need,
+// possibly impacting PartialOrd performance on top of being unnecessary and occuring in generated code.
+// Possibly the derivative crate could get updated to suppress this lint
+// from within their proc macros itself. Issue: https://github.com/mcarton/rust-derivative/issues/115
+#![allow(clippy::incorrect_partial_ord_impl_on_ord_type)]
+
 pub use error::*;
 pub mod error;
 

crypto/rust/src/chain_crypto/digest.rs

@@ -337,7 +337,7 @@ impl<H: DigestAlg, T> Eq for DigestOf<H, T> {}
 
 impl<H: DigestAlg, T> PartialOrd for DigestOf<H, T> {
     fn partial_cmp(&self, other: &Self) -> Option<std::cmp::Ordering> {
-        self.inner.partial_cmp(&other.inner)
+        Some(self.cmp(other))
     }
 }
 

crypto/rust/src/chain_crypto/key.rs

@@ -273,7 +273,7 @@ impl<A: AsymmetricPublicKey> std::cmp::Eq for PublicKey<A> {}
 
 impl<A: AsymmetricPublicKey> std::cmp::PartialOrd<Self> for PublicKey<A> {
     fn partial_cmp(&self, other: &Self) -> Option<std::cmp::Ordering> {
-        self.0.as_ref().partial_cmp(other.0.as_ref())
+        Some(self.cmp(other))
     }
 }
 

crypto/rust/src/lib.rs

@@ -478,7 +478,7 @@ macro_rules! impl_signature {
 
         impl PartialOrd for $name {
             fn partial_cmp(&self, other: &Self) -> Option<std::cmp::Ordering> {
-                self.0.as_ref().partial_cmp(other.0.as_ref())
+                Some(self.cmp(other))
             }
         }