Merge pull request #2463 from suchapalaver/fix/partial-ord-implementation-for-prune-candidate

vaporos · web-flow · commit 4f6994e78122 · 2023-11-20T11:26:04.000-05:00
Fix PartialOrd implementation for prune candidate
diff --git a/validator/src/state/state_pruning_manager.rs b/validator/src/state/state_pruning_manager.rs
@@ -49,7 +49,7 @@ impl Ord for PruneCandidate {
 
 impl PartialOrd for PruneCandidate {
     fn partial_cmp(&self, other: &PruneCandidate) -> Option<Ordering> {
-        Some(Ordering::reverse(self.0.cmp(&other.0)))
+        Some(self.cmp(other))
     }
 }
 

Original file line number	Diff line number	Diff line change
`@@ -49,7 +49,7 @@ impl Ord for PruneCandidate {`
`49`	`49`
`50`	`50`	`impl PartialOrd for PruneCandidate {`
`51`	`51`	`fn partial_cmp(&self, other: &PruneCandidate) -> Option<Ordering> {`
`52`		`- Some(Ordering::reverse(self.0.cmp(&other.0)))`
	`52`	`+ Some(self.cmp(other))`
`53`	`53`	`}`
`54`	`54`	`}`
`55`	`55`