Skip to content

Commit cfb2305

Browse files
author
Matt Goldberg
committed
fix: align CardPlay ordering semantics
1 parent b728136 commit cfb2305

File tree

1 file changed

+1
-13
lines changed

1 file changed

+1
-13
lines changed

types/src/card_play.rs

Lines changed: 1 addition & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -34,19 +34,7 @@ impl Ord for CardPlay {
3434
if std::mem::discriminant(self) != std::mem::discriminant(other) {
3535
panic!("Cannot compare CardPlay variants of different types");
3636
}
37-
let our_card = match self {
38-
CardPlay::Single(card) => card,
39-
CardPlay::Pair(card, _) => card,
40-
CardPlay::Triple(card, _, _) => card,
41-
CardPlay::Quad(card, _, _, _) => card,
42-
};
43-
let their_card = match other {
44-
CardPlay::Single(card) => card,
45-
CardPlay::Pair(card, _) => card,
46-
CardPlay::Triple(card, _, _) => card,
47-
CardPlay::Quad(card, _, _, _) => card,
48-
};
49-
our_card.cmp(their_card)
37+
self.value().cmp(&other.value())
5038
}
5139
}
5240

0 commit comments

Comments
 (0)