Change frexp10 overloads

Author: mborland

include/boost/decimal/decimal128.hpp

@@ -575,10 +575,7 @@ BOOST_DECIMAL_EXPORT class decimal128 final
 
     // <cmath> functions that need to be friends
     template <BOOST_DECIMAL_DECIMAL_FLOATING_TYPE T>
-    friend constexpr auto frexp10(T num, int* expptr) noexcept
-    -> std::enable_if_t<detail::is_decimal_floating_point_v<T>,
-            std::conditional_t<std::is_same<T, decimal32>::value, std::uint32_t,
-                    std::conditional_t<std::is_same<T, decimal64>::value, std::uint64_t, detail::uint128>>>;
+    friend constexpr auto frexp10(T num, int* expptr) noexcept;
 
     friend constexpr auto copysignd128(decimal128 mag, decimal128 sgn) noexcept -> decimal128;
     friend constexpr auto scalblnd128(decimal128 num, long exp) noexcept -> decimal128;

include/boost/decimal/decimal32.hpp

@@ -561,10 +561,7 @@ BOOST_DECIMAL_EXPORT class decimal32 final // NOLINT(cppcoreguidelines-special-m
 
     // Related to <cmath>
     template <BOOST_DECIMAL_DECIMAL_FLOATING_TYPE T>
-    friend constexpr auto frexp10(T num, int* expptr) noexcept
-    -> std::enable_if_t<detail::is_decimal_floating_point_v<T>,
-            std::conditional_t<std::is_same<T, decimal32>::value, std::uint32_t,
-                    std::conditional_t<std::is_same<T, decimal64>::value, std::uint64_t, detail::uint128>>>;
+    friend constexpr auto frexp10(T num, int* expptr) noexcept;
 
     friend constexpr auto scalbnd32(decimal32 num, int exp) noexcept -> decimal32;
     friend constexpr auto scalblnd32(decimal32 num, long exp) noexcept -> decimal32;

include/boost/decimal/decimal32_fast.hpp

@@ -64,6 +64,9 @@ class decimal32_fast final
     friend constexpr auto to_integral(Decimal val) noexcept
         BOOST_DECIMAL_REQUIRES_TWO_RETURN(detail::is_decimal_floating_point_v, Decimal, detail::is_integral_v, TargetType, TargetType);
 
+    template <BOOST_DECIMAL_DECIMAL_FLOATING_TYPE T>
+    friend constexpr auto frexp10(T num, int* expptr) noexcept;
+
 public:
     constexpr decimal32_fast() noexcept : significand_{}, exponent_{}, sign_ {} {}
 
@@ -135,31 +138,7 @@ class decimal32_fast final
     explicit constexpr operator detail::int128_t() const noexcept;
     explicit constexpr operator detail::uint128_t() const noexcept;
     #endif
-
-    #if !defined(BOOST_DECIMAL_DISABLE_CLIB)
-
-    // TODO(mborland): Remove and use the base implementation in io.hpp
-    template <typename charT, typename traits>
-    friend auto operator<<(std::basic_ostream<charT, traits>& os, const decimal32_fast& d) -> std::basic_ostream<charT, traits>&
-    {
-        if (d.sign_)
-        {
-            os << '-';
-        }
-
-        os << d.significand_ << "e";
-        const auto biased_exp {d.biased_exponent()};
-        if (biased_exp > 0)
-        {
-            os << '+';
-        }
-        os << biased_exp;
-
-        return os;
-    }
-
-    #endif
-
+    
     friend constexpr auto direct_init(std::uint32_t significand, std::uint8_t exponent, bool sign) noexcept -> decimal32_fast;
 };
 

include/boost/decimal/decimal64.hpp

@@ -563,10 +563,7 @@ BOOST_DECIMAL_EXPORT class decimal64 final
 
     // <cmath> functions that need to be friends
     template <BOOST_DECIMAL_DECIMAL_FLOATING_TYPE T>
-    friend constexpr auto frexp10(T num, int* expptr) noexcept
-    -> std::enable_if_t<detail::is_decimal_floating_point_v<T>,
-            std::conditional_t<std::is_same<T, decimal32>::value, std::uint32_t,
-                    std::conditional_t<std::is_same<T, decimal64>::value, std::uint64_t, detail::uint128>>>;
+    friend constexpr auto frexp10(T num, int* expptr) noexcept;
 
     friend constexpr auto copysignd64(decimal64 mag, decimal64 sgn) noexcept -> decimal64;
     friend constexpr auto fmad64(decimal64 x, decimal64 y, decimal64 z) noexcept -> decimal64;

include/boost/decimal/detail/cmath/frexp10.hpp

@@ -26,12 +26,9 @@ namespace decimal {
 // If the conversion can not be performed returns UINT32_MAX and exp = 0
 BOOST_DECIMAL_EXPORT template <BOOST_DECIMAL_DECIMAL_FLOATING_TYPE T>
 constexpr auto frexp10(T num, int* expptr) noexcept
-    -> std::enable_if_t<detail::is_decimal_floating_point_v<T>,
-           std::conditional_t<std::is_same<T, decimal32>::value, std::uint32_t,
-                std::conditional_t<std::is_same<T, decimal64>::value, std::uint64_t, detail::uint128>>>
 {
-    using ReturnType = std::conditional_t<std::is_same<T, decimal32>::value, std::uint32_t,
-                           std::conditional_t<std::is_same<T, decimal64>::value, std::uint64_t, detail::uint128>>;
+    using ReturnType = std::conditional_t<std::is_same<T, decimal32>::value || std::is_same<T, decimal32_fast>::value, std::uint32_t,
+                       std::conditional_t<std::is_same<T, decimal64>::value, std::uint64_t, detail::uint128>>;
 
     constexpr T zero {0, 0};