Index: include/variant =================================================================== --- include/variant +++ include/variant @@ -1098,11 +1098,39 @@ template struct __overload<_Tp, _Types...> : __overload<_Types...> { using __overload<_Types...>::operator(); - __identity<_Tp> operator()(_Tp) const; + + static auto __test(_Tp (&&)[1]) -> __identity<_Tp>; + + template + auto operator()(_Tp, _Up&& __t) const + -> decltype(__test({ _VSTD::forward<_Up>(__t) })); }; +template +struct __overload_bool : _Base { + using _Base::operator(); + + template > + auto operator()(bool, _Up&&) const + -> enable_if_t, __identity<_Tp>>; +}; + +template +struct __overload + : __overload_bool<__overload<_Types...>, bool> {}; +template +struct __overload + : __overload_bool<__overload<_Types...>, bool const> {}; +template +struct __overload + : __overload_bool<__overload<_Types...>, bool volatile> {}; +template +struct __overload + : __overload_bool<__overload<_Types...>, bool const volatile> {}; + template -using __best_match_t = typename result_of_t<__overload<_Types...>(_Tp&&)>::type; +using __best_match_t = + typename invoke_result_t<__overload<_Types...>, _Tp, _Tp>::type; } // __variant_detail Index: test/std/utilities/variant/variant.variant/variant.assign/T.pass.cpp =================================================================== --- test/std/utilities/variant/variant.variant/variant.assign/T.pass.cpp +++ test/std/utilities/variant/variant.variant/variant.assign/T.pass.cpp @@ -22,6 +22,7 @@ #include #include #include +#include #include "test_macros.h" #include "variant_test_helpers.hpp" @@ -122,7 +123,7 @@ void test_T_assignment_sfinae() { { - using V = std::variant; + using V = std::variant; static_assert(!std::is_assignable::value, "ambiguous"); } { @@ -133,6 +134,31 @@ using V = std::variant; static_assert(!std::is_assignable::value, "no matching operator="); } + { + using V = std::variant; + static_assert(!std::is_assignable::value, "no matching operator="); + } + { + using V = std::variant, bool>; + static_assert(!std::is_assignable>::value, + "no explicit bool in operator="); + struct X { + operator void*(); + }; + static_assert(!std::is_assignable::value, + "no boolean conversion in operator="); + static_assert(!std::is_assignable::value, + "no converted to bool in operator="); + } + { + struct X {}; + struct Y { + operator X(); + }; + using V = std::variant; + static_assert(std::is_assignable::value, + "regression on user-defined conversions in operator="); + } #if !defined(TEST_VARIANT_HAS_NO_REFERENCES) { using V = std::variant; @@ -161,6 +187,37 @@ assert(v.index() == 1); assert(std::get<1>(v) == 43); } + { + std::variant v; + v = 42; + assert(v.index() == 1); + assert(std::get<1>(v) == 42); + v = 43u; + assert(v.index() == 0); + assert(std::get<0>(v) == 43); + } + { + std::variant v = true; + v = "bar"; + assert(v.index() == 0); + assert(std::get<0>(v) == "bar"); + } + { + std::variant> v; + v = nullptr; + assert(v.index() == 1); + assert(std::get<1>(v) == nullptr); + } + { + std::variant v = 42; + v = false; + assert(v.index() == 0); + assert(!std::get<0>(v)); + bool lvt = true; + v = lvt; + assert(v.index() == 0); + assert(std::get<0>(v)); + } #if !defined(TEST_VARIANT_HAS_NO_REFERENCES) { using V = std::variant; Index: test/std/utilities/variant/variant.variant/variant.ctor/T.pass.cpp =================================================================== --- test/std/utilities/variant/variant.variant/variant.ctor/T.pass.cpp +++ test/std/utilities/variant/variant.variant/variant.ctor/T.pass.cpp @@ -20,8 +20,8 @@ #include #include #include +#include -#include "test_convertible.hpp" #include "test_macros.h" #include "variant_test_helpers.hpp" @@ -39,6 +39,8 @@ struct AnyConstructible { template AnyConstructible(T&&) {} }; struct NoConstructible { NoConstructible() = delete; }; +template +struct RValueConvertibleFrom { RValueConvertibleFrom(T&&) {} }; void test_T_ctor_noexcept() { { @@ -53,7 +55,7 @@ void test_T_ctor_sfinae() { { - using V = std::variant; + using V = std::variant; static_assert(!std::is_constructible::value, "ambiguous"); } { @@ -66,6 +68,32 @@ "no matching constructor"); } { + using V = std::variant; + static_assert(!std::is_constructible::value, + "no matching constructor"); + } + { + using V = std::variant, bool>; + static_assert(!std::is_constructible>::value, + "no explicit bool in constructor"); + struct X { + operator void*(); + }; + static_assert(!std::is_constructible::value, + "no boolean conversion in constructor"); + static_assert(!std::is_constructible::value, + "no converted to bool in constructor"); + } + { + struct X {}; + struct Y { + operator X(); + }; + using V = std::variant; + static_assert(std::is_constructible::value, + "regression on user-defined conversions in constructor"); + } + { using V = std::variant; static_assert( !std::is_constructible>::value, @@ -99,6 +127,34 @@ static_assert(v.index() == 1, ""); static_assert(std::get<1>(v) == 42, ""); } + { + constexpr std::variant v(42); + static_assert(v.index() == 1, ""); + static_assert(std::get<1>(v) == 42, ""); + } + { + std::variant v = "foo"; + assert(v.index() == 0); + assert(std::get<0>(v) == "foo"); + } + { + std::variant> v = nullptr; + assert(v.index() == 1); + assert(std::get<1>(v) == nullptr); + } + { + std::variant v = true; + assert(v.index() == 0); + assert(std::get<0>(v)); + } + { + std::variant> v1 = 42; + assert(v1.index() == 0); + + int x = 42; + std::variant, AnyConstructible> v2 = x; + assert(v2.index() == 1); + } #if !defined(TEST_VARIANT_HAS_NO_REFERENCES) { using V = std::variant; Index: www/cxx2a_status.html =================================================================== --- www/cxx2a_status.html +++ www/cxx2a_status.html @@ -115,7 +115,7 @@ P0591R4LWGUtility functions to implement uses-allocator constructionSan Diego P0595R2CWGP0595R2 std::is_constant_evaluated()San DiegoComplete9.0 P0602R4LWGvariant and optional should propagate copy/move trivialitySan DiegoComplete8.0 - P0608R3LWGA sane variant converting constructorSan Diego + P0608R3LWGA sane variant converting constructorSan DiegoComplete9.0 P0655R1LWGvisit<R>: Explicit Return Type for visitSan Diego P0771R1LWGstd::function move constructor should be noexceptSan DiegoComplete6.0 P0896R4LWGThe One Ranges ProposalSan Diego