Fix 'PruneCandidate' 'PartialOrd' impl to return result of 'self.cmp(other)', not its reverse

suchapalaver · suchapalaver · commit 2ff4ca2afb92 · 2023-10-25T17:48:21.000-04:00
Signed-off-by: Joseph Livesey &lt;jlivesey@gmail.com&gt;
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`