From 02c44c92d0c6c083adb92ad0e61536d40d7b844e Mon Sep 17 00:00:00 2001
From: steven-omaha <35634100+steven-omaha@users.noreply.github.com>
Date: Tue, 10 Oct 2023 15:28:28 +0200
Subject: [PATCH] fix(grouping): non-canonical ordering

clippy: incorrect_partial_ord_impl_on_ord_type
---
 crates/pacdef_core/src/grouping/group.rs   | 5 +----
 crates/pacdef_core/src/grouping/section.rs | 2 +-
 2 files changed, 2 insertions(+), 5 deletions(-)

diff --git a/crates/pacdef_core/src/grouping/group.rs b/crates/pacdef_core/src/grouping/group.rs
index 8ea0c7b..c8a0d3a 100644
--- a/crates/pacdef_core/src/grouping/group.rs
+++ b/crates/pacdef_core/src/grouping/group.rs
@@ -94,10 +94,7 @@ fn is_child_of_any_dir(path: &Path, dirs: &[PathBuf]) -> bool {
 
 impl PartialOrd for Group {
     fn partial_cmp(&self, other: &Self) -> Option<std::cmp::Ordering> {
-        match self.name.partial_cmp(&other.name) {
-            Some(core::cmp::Ordering::Equal) => None,
-            ord => ord,
-        }
+        Some(self.cmp(other))
     }
 }
 
diff --git a/crates/pacdef_core/src/grouping/section.rs b/crates/pacdef_core/src/grouping/section.rs
index 00ad032..b79218a 100644
--- a/crates/pacdef_core/src/grouping/section.rs
+++ b/crates/pacdef_core/src/grouping/section.rs
@@ -67,7 +67,7 @@ impl Eq for Section {
 
 impl PartialOrd for Section {
     fn partial_cmp(&self, other: &Self) -> Option<std::cmp::Ordering> {
-        self.name.partial_cmp(&other.name)
+        Some(self.cmp(other))
     }
 }