Thanks for fixing this! On Mon, Apr 1, 2019 at 11:14 PM Reid Kleckner via cfe-commits < cfe-commits@lists.llvm.org> wrote:
> Author: rnk > Date: Mon Apr 1 14:16:17 2019 > New Revision: 357429 > > URL: http://llvm.org/viewvc/llvm-project?rev=357429&view=rev > Log: > Fix clangd unittest _WIN32 ifdef > > WIN32 is not defined, _WIN32 is, use that instead. > > Modified: > clang-tools-extra/trunk/unittests/clangd/JSONTransportTests.cpp > > Modified: clang-tools-extra/trunk/unittests/clangd/JSONTransportTests.cpp > URL: > http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/unittests/clangd/JSONTransportTests.cpp?rev=357429&r1=357428&r2=357429&view=diff > > ============================================================================== > --- clang-tools-extra/trunk/unittests/clangd/JSONTransportTests.cpp > (original) > +++ clang-tools-extra/trunk/unittests/clangd/JSONTransportTests.cpp Mon > Apr 1 14:16:17 2019 > @@ -17,8 +17,8 @@ namespace { > > // No fmemopen on windows or on versions of MacOS X earlier than 10.13, > so we > // can't easily run this test. > -#if !(defined(WIN32) || (defined(__MAC_OS_X_VERSION_MIN_REQUIRED) && > \ > - __MAC_OS_X_VERSION_MIN_REQUIRED < 101300)) > +#if !(defined(_WIN32) || (defined(__MAC_OS_X_VERSION_MIN_REQUIRED) && > \ > + __MAC_OS_X_VERSION_MIN_REQUIRED < 101300)) > > // Fixture takes care of managing the input/output buffers for the > transport. > class JSONTransportTest : public ::testing::Test { > > > _______________________________________________ > cfe-commits mailing list > cfe-commits@lists.llvm.org > https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits > -- Regards, Ilya Biryukov
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
