Made replace_ignored a new format callable (#2061)

jfalcou · Feb 10, 2025 · 0762239 · 0762239
1 parent 0f88013
commit 0762239
Showing 1 changed file with 53 additions and 48 deletions.
diff --git a/include/eve/module/core/regular/replace.hpp b/include/eve/module/core/regular/replace.hpp
@@ -15,56 +15,61 @@
 
 namespace eve
 {
-// TODO DOC is it to be in user interface ?
-//================================================================================================
-//! @addtogroup core_logical
-//! @{
-//!    @var replace_ignored
-//!    @brief A small helper to replace ignored values
-//!
-//!   @groupheader{Header file}
-//!   @code
-//!   #include <eve/module/core.hpp>
-//!   @endcode
-//!
-//!   A convinience wrapper around if_else.
-//!
-//!   @groupheader{Callable Signatures}
-//!
-//!   @code
-//!   namespace eve
-//!   {
-//!      template< eve::value T, eve::conditional_expr Ignore, eve::value Other >
-//!      auto replace_ignored(T x, Ignore ignore, Other with )
-//!        -> decltype(eve::if_else(ignore, x, with))
-//!   }
-//!   @endcode
-//!
-//!   **Parameters**
-//!
-//!   * `x` - main value
-//!   * `ignore` - selection
-//!   * `with` - value to replace with
-//!
-//!   **Return value**
-//!
-//!   Same as if_else.
-//!
-//================================================================================================
+  template<typename Options>
+  struct replace_ignored_t : callable<replace_ignored_t, Options>
+  {
+    template<simd_value T, relative_conditional_expr Ignore, value Other>
+    constexpr EVE_FORCEINLINE T operator()(T x, Ignore ignore, Other with) const noexcept
+    {
+      return eve::if_else(ignore, x, with);
+    }
 
-EVE_MAKE_CALLABLE(replace_ignored_, replace_ignored);
+    template<simd_value T, relative_conditional_expr Ignore, generator Other>
+    constexpr EVE_FORCEINLINE T operator()(T x, Ignore ignore, Other with) const noexcept
+    {
+      return eve::if_else(ignore, x, with);
+    }
+  };
 
-namespace detail
-{
   //================================================================================================
-  // Helper function - Replace ignored value with an alternative
+  //! @addtogroup core_logical
+  //! @{
+  //!    @var replace_ignored
+  //!    @brief A small helper to replace ignored values
+  //!
+  //!   @groupheader{Header file}
+  //!   @code
+  //!   #include <eve/module/core.hpp>
+  //!   @endcode
+  //!
+  //!   A convinience wrapper around if_else.
+  //!
+  //!   @groupheader{Callable Signatures}
+  //!
+  //!   @code
+  //!   namespace eve
+  //!   {
+  //!      template<simd_value T, relative_conditional_expr Ignore, value Other>
+  //!      T replace_ignored(T x, Ignore ignore, Other with);
+  //!
+  //!      template<simd_value T, relative_conditional_expr Ignore, generator Other>
+  //!      T replace_ignored(T x, Ignore ignore, Other with);
+  //!   }
+  //!   @endcode
+  //!
+  //!   **Parameters**
+  //!
+  //!   * `x` - main value
+  //!   * `ignore` - selection
+  //!   * `with` - value to replace with
+  //!
+  //!   **Return value**
+  //!
+  //!   Same as if_else.
+  //!
+  //================================================================================================
+  inline constexpr auto replace_ignored = functor<replace_ignored_t>;
+  //================================================================================================
+  //! @}
   //================================================================================================
-  template<simd_value Wide, conditional_expr Ignore, typename Other>
-  EVE_FORCEINLINE auto
-  replace_ignored_(EVE_SUPPORTS(cpu_), Wide x, Ignore ignore, Other with) noexcept
-      -> decltype(eve::if_else(ignore, x, with))
-  {
-    return eve::if_else(ignore, x, with);
-  }
-}
 }