Consistent ImmutableDeserializedPacket::PartialEq (#3098)

anza-xyz · Oct 8, 2024 · 10b447f · 10b447f
1 parent f1b3856
commit 10b447f
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/core/src/banking_stage/immutable_deserialized_packet.rs b/core/src/banking_stage/immutable_deserialized_packet.rs
@@ -39,7 +39,7 @@ pub enum DeserializedPacketError {
     FailedFilter(#[from] PacketFilterFailure),
 }
 
-#[derive(Debug, PartialEq, Eq)]
+#[derive(Debug, Eq)]
 pub struct ImmutableDeserializedPacket {
     original_packet: Packet,
     transaction: SanitizedVersionedTransaction,
@@ -132,6 +132,13 @@ impl ImmutableDeserializedPacket {
     }
 }
 
+// PartialEq MUST be consistent with PartialOrd and Ord
+impl PartialEq for ImmutableDeserializedPacket {
+    fn eq(&self, other: &Self) -> bool {
+        self.compute_unit_price() == other.compute_unit_price()
+    }
+}
+
 impl PartialOrd for ImmutableDeserializedPacket {
     fn partial_cmp(&self, other: &Self) -> Option<Ordering> {
         Some(self.cmp(other))