Don't run the thread local tests requiring dlopen on Windows
authorChristopher Dykes <cdykes@fb.com>
Tue, 26 Jul 2016 22:53:04 +0000 (15:53 -0700)
committerFacebook Github Bot 1 <facebook-github-bot-1-bot@fb.com>
Tue, 26 Jul 2016 23:08:32 +0000 (16:08 -0700)
Summary: When I initially added the surrounding `#ifdef` I failed at where I put my `#endif`, as I left the shared library test enabled. This disables it for Windows.

Reviewed By: yfeldblum

Differential Revision: D3622769

fbshipit-source-id: f263201239095bbef076300d5f40649ebee1173c

folly/test/ThreadLocalTest.cpp

index e836ef37d4290a4b2fe21f11a36a3e0f41494768..e2e2fce0ead5a8dced6088e8830eb604f5c86f91 100644 (file)
@@ -575,7 +575,6 @@ TEST(ThreadLocal, Fork2) {
     EXPECT_TRUE(false) << "fork failed";
   }
 }
-#endif
 
 // Elide this test when using any sanitizer. Otherwise, the dlopen'ed code
 // would end up running without e.g., ASAN-initialized data structures and
@@ -624,6 +623,7 @@ TEST(ThreadLocal, SharedLibrary) {
   t2.join();
 }
 
+#endif
 #endif
 
 namespace folly { namespace threadlocal_detail {