[libcxx-commits] [libcxx] r366025 - Harden variant test added in r366022

Eric Fiselier via libcxx-commits libcxx-commits at lists.llvm.org
Sun Jul 14 11:30:34 PDT 2019


Author: ericwf
Date: Sun Jul 14 11:30:34 2019
New Revision: 366025

URL: http://llvm.org/viewvc/llvm-project?rev=366025&view=rev
Log:
Harden variant test added in r366022

The test was brittle since it only went boom for one specific type, when
really it should go boom for all of them.

Modified:
    libcxx/trunk/test/std/utilities/variant/variant.variant/variant.ctor/T.pass.cpp

Modified: libcxx/trunk/test/std/utilities/variant/variant.variant/variant.ctor/T.pass.cpp
URL: http://llvm.org/viewvc/llvm-project/libcxx/trunk/test/std/utilities/variant/variant.variant/variant.ctor/T.pass.cpp?rev=366025&r1=366024&r2=366025&view=diff
==============================================================================
--- libcxx/trunk/test/std/utilities/variant/variant.variant/variant.ctor/T.pass.cpp (original)
+++ libcxx/trunk/test/std/utilities/variant/variant.variant/variant.ctor/T.pass.cpp Sun Jul 14 11:30:34 2019
@@ -177,13 +177,13 @@ void test_T_ctor_basic() {
 #endif
 }
 
-struct BoomOnInt {
+struct BoomOnAnything {
   template <class T>
-  constexpr BoomOnInt(T) { static_assert(!std::is_same<T, int>::value, ""); }
+  constexpr BoomOnAnything(T) { static_assert(!std::is_same<T, T>::value, ""); }
 };
 
 void test_no_narrowing_check_for_class_types() {
-  using V = std::variant<int, BoomOnInt>;
+  using V = std::variant<int, BoomOnAnything>;
   V v(42);
   assert(v.index() == 0);
   assert(std::get<0>(v) == 42);




More information about the libcxx-commits mailing list