Merge pull request #2275 from JohanMabille/layout

Moved layout_remove_any to xlayout.hpp

Author: JohanMabille

include/xtensor/xeval.hpp

@@ -50,15 +50,6 @@ namespace xt
 
     namespace detail
     {
-        /************************************
-         * layout_remove_any implementation *
-         ************************************/
-
-        constexpr layout_type layout_remove_any(const layout_type layout)
-        {
-            return layout == layout_type::any ? XTENSOR_DEFAULT_LAYOUT : layout;
-        }
-
         /**********************************
          * has_same_layout implementation *
          **********************************/
@@ -102,7 +93,7 @@ namespace xt
          ****************************************/
 
         template <class E, layout_type L>
-        using as_xarray_container_t = xarray<typename std::decay_t<E>::value_type, detail::layout_remove_any(L)>;
+        using as_xarray_container_t = xarray<typename std::decay_t<E>::value_type, layout_remove_any(L)>;
 
         /*****************************************
          * as_xtensor_container_t implementation *
@@ -111,7 +102,7 @@ namespace xt
         template <class E, layout_type L>
         using as_xtensor_container_t = xtensor<typename std::decay_t<E>::value_type,
                                                std::tuple_size<typename std::decay_t<E>::shape_type>::value,
-                                               detail::layout_remove_any(L)>;
+                                               layout_remove_any(L)>;
     }
 
     /**

include/xtensor/xlayout.hpp

@@ -52,6 +52,8 @@ namespace xt
 
     constexpr layout_type default_assignable_layout(layout_type l) noexcept;
 
+    constexpr layout_type layout_remove_any(const layout_type layout) noexcept;
+
     /******************
      * Implementation *
      ******************/
@@ -92,6 +94,11 @@ namespace xt
         return (l == layout_type::row_major || l == layout_type::column_major) ?
             l : XTENSOR_DEFAULT_LAYOUT;
     }
+
+    constexpr layout_type layout_remove_any(const layout_type layout) noexcept
+    {
+        return layout == layout_type::any ? XTENSOR_DEFAULT_LAYOUT : layout;
+    }
 }
 
 #endif