From 033235df8c80e715ee7423e0b7977cbb239b275e Mon Sep 17 00:00:00 2001
From: Jonathan Wakely <cxx@kayari.org>
Date: Mon, 4 Nov 2024 17:46:03 +0000
Subject: [PATCH] Use C++20 default comparisons for section objects

---
 src/sections.cpp | 32 +-------------------------------
 src/sections.h   | 17 ++++-------------
 2 files changed, 5 insertions(+), 44 deletions(-)

diff --git a/src/sections.cpp b/src/sections.cpp
index 62f8a5f0a4..b3f8f10c40 100644
--- a/src/sections.cpp
+++ b/src/sections.cpp
@@ -4,22 +4,9 @@
 #include <sstream>
 #include <iostream>
 #include <cctype>
+#include <tuple>
 #include <utility>
 
-auto lwg::operator<(section_tag const & x, section_tag const & y) noexcept -> bool {
-   return (x.prefix < y.prefix) ?  true
-        : (y.prefix < x.prefix) ? false
-        : x.name < y.name;
-}
-
-auto lwg::operator==(section_tag const & x, section_tag const & y) noexcept -> bool {
-   return x.prefix == y.prefix && x.name == y.name;
-}
-
-auto lwg::operator!=(section_tag const & x, section_tag const & y) noexcept -> bool {
-   return !(x == y);
-}
-
 auto lwg::operator << (std::ostream& os, section_tag const & tag) -> std::ostream & {
   os << '[';
    if (!tag.prefix.empty()) { os << tag.prefix << "::"; }
@@ -34,23 +21,6 @@ std::string lwg::as_string(section_tag const & x)
     : x.prefix + "::" + x.name;
 }
 
-auto lwg::operator < (section_num const & x, section_num const & y) noexcept -> bool {
-   // prefixes are unique, so there should be no need for a tiebreak.
-   return (x.prefix < y.prefix) ?  true
-        : (y.prefix < x.prefix) ? false
-        : x.num < y.num;
-}
-
-auto lwg::operator == (section_num const & x, section_num const & y) noexcept -> bool {
-   return (x.prefix != y.prefix)
-        ? false
-        : x.num == y.num;
-}
-
-auto lwg::operator != (section_num const & x, section_num const & y) noexcept -> bool {
-   return !(x == y);
-}
-
 auto lwg::operator >> (std::istream& is, section_num& sn) -> std::istream & {
    sn.prefix.clear();
    sn.num.clear();
diff --git a/src/sections.h b/src/sections.h
index f9727afa6e..53e8b9de94 100644
--- a/src/sections.h
+++ b/src/sections.h
@@ -13,33 +13,24 @@ struct section_tag
 {
   std::string  prefix;        // example: fund.ts.v2 
   std::string  name;          // example: meta.logical
+  bool operator==(const section_tag&) const = default;
+  auto operator<=>(const section_tag&) const = default;
 };
 
 struct section_num {
   std::string       prefix;    // example: fund.ts.v2 
   std::vector<int>  num;       // sequence of numbers corresponding to section number
                                // in relevant doc, e.g,, 17.5.2.1.4.2
+  bool operator==(const section_num&) const = default;
+  auto operator<=>(const section_num&) const = default;
 };
 
 using section_map = std::map<section_tag, section_num>;
 
-auto operator <  (section_tag const & x, section_tag const & y) noexcept -> bool;
-auto operator == (section_tag const & x, section_tag const & y) noexcept -> bool;
-auto operator != (section_tag const & x, section_tag const & y) noexcept -> bool;
 auto operator << (std::ostream & os,
   section_tag const & tag) -> std::ostream &; // with square brackets
 std::string as_string(section_tag const & x); // without square brackets
 
-auto operator <  (section_num const & x, section_num const & y) noexcept -> bool;
-   // section 'x' sorts before section 'y' if its 'prefix' field lexicographically
-   // precedes that of 'y',  and its 'nun' field lexicographically precedes that
-   // of 'y' if the prefix fields are equivalent.
-
-auto operator == (section_num const & x, section_num const & y) noexcept -> bool;
-auto operator != (section_num const & x, section_num const & y) noexcept -> bool;
-   // Two 'section_num' objects compare equal if their 'prefix' and 'num' both
-   // compare equal.
-
 auto operator >> (std::istream & is, section_num & sn) -> std::istream &;
 auto operator << (std::ostream & os, section_num const & sn) -> std::ostream &;