[clang-tools-extra] r251239 - [clang-tidy] Fix for build bots not liking #include <cstddef>

Piotr Dziwinski via cfe-commits cfe-commits at lists.llvm.org
Sun Oct 25 08:47:21 PDT 2015


Author: piotrdz
Date: Sun Oct 25 10:47:21 2015
New Revision: 251239

URL: http://llvm.org/viewvc/llvm-project?rev=251239&view=rev
Log:
[clang-tidy] Fix for build bots not liking #include <cstddef>

Modified:
    clang-tools-extra/trunk/test/clang-tidy/readability-implicit-bool-cast-cxx98.cpp
    clang-tools-extra/trunk/test/clang-tidy/readability-implicit-bool-cast.cpp

Modified: clang-tools-extra/trunk/test/clang-tidy/readability-implicit-bool-cast-cxx98.cpp
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/clang-tidy/readability-implicit-bool-cast-cxx98.cpp?rev=251239&r1=251238&r2=251239&view=diff
==============================================================================
--- clang-tools-extra/trunk/test/clang-tidy/readability-implicit-bool-cast-cxx98.cpp (original)
+++ clang-tools-extra/trunk/test/clang-tidy/readability-implicit-bool-cast-cxx98.cpp Sun Oct 25 10:47:21 2015
@@ -1,6 +1,9 @@
 // RUN: clang-tidy %s -checks=-*,readability-implicit-bool-cast -- -std=c++98
 
-#include <cstddef> // for NULL
+// We need NULL macro, but some buildbots don't like including <cstddef> header
+// This is a portable way of getting it to work
+#undef NULL
+#define NULL 0L
 
 template<typename T>
 void functionTaking(T);

Modified: clang-tools-extra/trunk/test/clang-tidy/readability-implicit-bool-cast.cpp
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/clang-tidy/readability-implicit-bool-cast.cpp?rev=251239&r1=251238&r2=251239&view=diff
==============================================================================
--- clang-tools-extra/trunk/test/clang-tidy/readability-implicit-bool-cast.cpp (original)
+++ clang-tools-extra/trunk/test/clang-tidy/readability-implicit-bool-cast.cpp Sun Oct 25 10:47:21 2015
@@ -1,6 +1,9 @@
 // RUN: %check_clang_tidy %s readability-implicit-bool-cast %t
 
-#include <cstddef> // for NULL
+// We need NULL macro, but some buildbots don't like including <cstddef> header
+// This is a portable way of getting it to work
+#undef NULL
+#define NULL 0L
 
 template<typename T>
 void functionTaking(T);




More information about the cfe-commits mailing list