Disable the CArray RangeFunc tests under MSVC
authorChristopher Dykes <cdykes@fb.com>
Wed, 16 Nov 2016 22:50:29 +0000 (14:50 -0800)
committerFacebook Github Bot <facebook-github-bot-bot@fb.com>
Wed, 16 Nov 2016 22:53:38 +0000 (14:53 -0800)
Summary: Because it won't compile due to an MSVC bug. See the link in the comment for more info.

Reviewed By: yfeldblum

Differential Revision: D4191233

fbshipit-source-id: 523deebe12379ff56934a4259b601d4ab677b9b6

folly/test/RangeTest.cpp

index c918a53c2c2a1f11f1e29bd87e18a38fc408aed1..5861b60f4ed29d31a5db0759affab4bc532abe75 100644 (file)
@@ -1093,10 +1093,14 @@ TEST(RangeFunc, Array) {
   testRangeFunc(x, 3);
 }
 
+// MSVC doesn't like it when you try to std::move C arrays:
+// https://developercommunity.visualstudio.com/content/problem/2441/
+#ifndef _MSC_VER
 TEST(RangeFunc, CArray) {
   int x[] {1, 2, 3, 4};
   testRangeFunc(x, 4);
 }
+#endif
 
 TEST(RangeFunc, ConstexprCArray) {
   static constexpr const int numArray[4] = {3, 17, 1, 9};