Implement Ord for consensus indices (MystenLabs#263)

Fix comments
mwtian · May 19, 2022 · a3078fe · a3078fe
1 parent 9c1a1e0
commit a3078fe
Showing 1 changed file with 10 additions and 11 deletions.
diff --git a/narwhal/executor/src/state.rs b/narwhal/executor/src/state.rs
@@ -65,17 +65,16 @@ impl ExecutionIndices {
 
 impl Ord for ExecutionIndices {
     fn cmp(&self, other: &Self) -> std::cmp::Ordering {
-        if self.next_certificate_index == other.next_certificate_index {
-            if self.next_batch_index == other.next_batch_index {
-                self.next_transaction_index
-                    .cmp(&other.next_transaction_index)
-            } else {
-                self.next_batch_index.cmp(&other.next_batch_index)
-            }
-        } else {
-            self.next_certificate_index
-                .cmp(&other.next_certificate_index)
-        }
+        (
+            self.next_certificate_index,
+            self.next_batch_index,
+            self.next_transaction_index,
+        )
+            .cmp(&(
+                other.next_certificate_index,
+                other.next_batch_index,
+                other.next_transaction_index,
+            ))
     }
 }