From b5932c453742cdddfb7214540c55c962eb244a25 Mon Sep 17 00:00:00 2001
From: Danny Robson <danny@nerdcruft.net>
Date: Mon, 12 Dec 2016 17:06:55 +1100
Subject: [PATCH] extent: prefer constexpr functions over static variables

---
 extent.cpp | 13 -------------
 extent.hpp |  4 ++--
 extent.ipp | 23 +++++++++++++++++++++++
 3 files changed, 25 insertions(+), 15 deletions(-)

diff --git a/extent.cpp b/extent.cpp
index 7c0b0b10..03b4becb 100644
--- a/extent.cpp
+++ b/extent.cpp
@@ -86,19 +86,6 @@ extent<S,T>::empty (void) const
 }
 
 
-///////////////////////////////////////////////////////////////////////////////
-template <size_t S, typename T>
-const extent<S,T> extent<S,T>::MIN { 0 };
-
-
-//-----------------------------------------------------------------------------
-template <size_t S, typename T>
-const extent<S,T> extent<S,T>::MAX
-{
-    std::numeric_limits<T>::max ()
-};
-
-
 ///////////////////////////////////////////////////////////////////////////////
 template <size_t S, typename T>
 extent_range<S,T>::extent_range (extent<S,T> _target):
diff --git a/extent.hpp b/extent.hpp
index e983d74a..eeb804a6 100644
--- a/extent.hpp
+++ b/extent.hpp
@@ -51,8 +51,8 @@ namespace util {
 
         bool empty (void) const;
 
-        static const extent MAX;
-        static const extent MIN;
+        static constexpr extent max (void);
+        static constexpr extent min (void);
     };
 
     template <size_t S, typename T>
diff --git a/extent.ipp b/extent.ipp
index 8c4ddecf..2f732d56 100644
--- a/extent.ipp
+++ b/extent.ipp
@@ -22,6 +22,7 @@
 #define __UTIL_EXTENT_IPP
 
 #include <algorithm>
+#include <limits>
 
 
 ///////////////////////////////////////////////////////////////////////////////
@@ -76,3 +77,25 @@ util::extent<S,T>::includes (point<S,U> p) const
             return false;
     return true;
 }
+
+
+///////////////////////////////////////////////////////////////////////////////
+template <size_t S, typename T>
+constexpr
+util::extent<S,T>
+util::extent<S,T>::max (void)
+{
+    return extent<S,T> {
+        std::numeric_limits<T>::max ()
+    };
+}
+
+
+//-----------------------------------------------------------------------------
+template <size_t S, typename T>
+constexpr
+util::extent<S,T>
+util::extent<S,T>::min (void)
+{
+    return extent<S,T> { 0 };
+}