Revert some edits to fix GCC build

Author: toroidal-code

include/child.hpp

@@ -73,40 +73,40 @@ class Child {
 
   // TODO: Look in to making these references instead of pointer returns
   /** @brief Get the Child's parent. */
-  constexpr Child *get_parent() noexcept { return parent; }
-  constexpr const Child *get_parent() const noexcept {
+  Child *get_parent() noexcept { return parent; }
+  const Child *get_parent() const noexcept {
     return const_cast<Child *>(parent);
   }
 
   template <class C>
-  constexpr C get_parent_as() noexcept {
+  C get_parent_as() noexcept {
     return static_cast<C>(get_parent());
   }
 
   /** @brief Set the Child's parent */
-  constexpr void set_parent(Child *parent) noexcept { this->parent = parent; }
-  constexpr void set_parent(const Child *parent) noexcept {
+  void set_parent(Child *parent) noexcept { this->parent = parent; }
+  void set_parent(const Child *parent) noexcept {
     this->parent = const_cast<Child *>(parent);
   }
 
   /*--------- Formatter helper functions -----------*/
   // Check to see if the tree has a printer
-  constexpr const bool has_formatter() noexcept;
+  const bool has_formatter() noexcept;
 
   // Get the printer from the tree
-  constexpr Formatters::BaseFormatter &get_formatter() noexcept;
+  Formatters::BaseFormatter &get_formatter() noexcept;
 
-  constexpr void set_printer(const Formatters::BaseFormatter &formatter) {
+  void set_printer(const Formatters::BaseFormatter &formatter) {
     this->formatter = &const_cast<Formatters::BaseFormatter &>(formatter);
   }
 
   /*--------- Primary member functions -------------*/
 
   /** @brief Get the status of the object (success/failure) */
-  constexpr const bool get_status() noexcept { return this->status; }
-  constexpr const bool get_status() const noexcept { return this->status; }
+  const bool get_status() noexcept { return this->status; }
+  const bool get_status() const noexcept { return this->status; }
 
-  constexpr void failed() noexcept;  // Report failure to the object.
+  void failed() noexcept;  // Report failure to the object.
 
   // Calculate the padding for printing this object
   std::string padding() noexcept;
@@ -120,7 +120,7 @@ class Child {
  * This is propogated up the parent/child tree, so that when a child object
  * fails, the parent object is immediately updated to reflect that as well.
  */
-constexpr inline void Child::failed() noexcept {
+inline void Child::failed() noexcept {
   this->status = false;
   // propogates the failure up the tree
   if (has_parent()) this->get_parent()->failed();
@@ -134,13 +134,13 @@ inline std::string Child::padding() noexcept {
   return has_parent() ? get_parent()->padding() + "  " : "";
 }
 
-constexpr inline const bool Child::has_formatter() noexcept {
+inline const bool Child::has_formatter() noexcept {
   if (this->formatter != nullptr) return true;
   if (!this->has_parent()) return false;  // base case;
   return parent->has_formatter();
 }
 
-constexpr inline Formatters::BaseFormatter &Child::get_formatter() noexcept {
+inline Formatters::BaseFormatter &Child::get_formatter() noexcept {
   if (this->formatter) return *formatter;
   if (!this->has_parent()) std::terminate();
   return parent->get_formatter();

include/let.hpp

@@ -13,11 +13,10 @@ class LetBase {
   bool delivered;
 
  public:
-  LetBase() noexcept : delivered(false) {}
+  constexpr LetBase() noexcept : delivered(false) {}
   LetBase(const LetBase &copy) = default;
-  constexpr void reset() noexcept { delivered = false; }
+  void reset() noexcept { delivered = false; }
   constexpr const bool has_result() noexcept { return this->delivered; }
-  constexpr const bool has_result() const noexcept { return this->delivered; }
 };
 
 template <typename T>
@@ -37,9 +36,7 @@ class Let : public LetBase {
     return result.operator->();
   }
 
-  constexpr T &operator*() & { return value(); }
-  constexpr const T &operator*() const & { return value(); }
-
+  T &operator*() & { return value(); }
   T &value()&;
 };
 

include/result.hpp

@@ -29,11 +29,11 @@ class Result {
   Result &operator=(Result &&) = default;
 
   /*--------- Status helper functions --------------*/
-  constexpr const bool get_status() noexcept { return value; }
-  constexpr const bool get_status() const noexcept { return value; }
+  const bool get_status() noexcept { return value; }
+  const bool get_status() const noexcept { return value; }
 
-  constexpr operator bool() noexcept { return this->get_status(); }
-  constexpr operator bool() const noexcept { return this->get_status(); }
+  operator bool() noexcept { return this->get_status(); }
+  operator bool() const noexcept { return this->get_status(); }
 
   /*--------- Message helper functions -------------*/
   const std::string get_message() noexcept { return message; }