minmax() is better named range(). Leave minmax in place, too

ABRG-Models · Nov 2, 2023 · c2f491b · c2f491b
1 parent c810b4c
commit c2f491b
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/morph/vec.h b/morph/vec.h
@@ -575,6 +575,13 @@ namespace morph {
             return morph::range<S>(*mm.first, *mm.second);
         }
 
+        //! Return the range of the vec (the min and max values of the vec)
+        morph::range<S> range() const
+        {
+            auto mm = std::minmax_element (this->begin(), this->end());
+            return morph::range<S>(*mm.first, *mm.second);
+        }
+
         //! Return true if any element is zero
         bool has_zero() const
         {

diff --git a/morph/vvec.h b/morph/vvec.h
@@ -711,7 +711,13 @@ namespace morph {
         //! pass 'true' as the template arg, then you can test for nans, and return the min/max of
         //! the rest of the numbers
         template<bool test_for_nans = false>
-        morph::range<S> minmax() const
+        morph::range<S> minmax() const { return this->range<test_for_nans>(); }
+
+        //! Return the range of values in the vvec (the min and max values). If you pass 'true' as
+        //! the template arg, then you can test for nans, and return the min/max of the rest of the
+        //! numbers
+        template<bool test_for_nans = false>
+        morph::range<S> range() const
         {
             morph::range<S> r;
             if constexpr (test_for_nans) {