[PATCH] D32914: Introduce Wzero-as-null-pointer-constant.

[Roman Lebedev via Phabricator via cfe-commits]

lebedev.ri added a comment.

And a lot of warnings from code using googletest, highlight:

  ../src/librawspeed/metadata/ColorFilterArrayTest.cpp:56:1: error: zero as null pointer constant [-Werror,-Wzero-as-null-pointer-constant]
  TEST(ColorFilterArrayTestBasic, Constructor) {
  ^
  googletest/googletest-src/googletest/include/gtest/gtest.h:2187:42: note: expanded from macro 'TEST'
  # define TEST(test_case_name, test_name) GTEST_TEST(test_case_name, test_name)
                                           ^
  googletest/googletest-src/googletest/include/gtest/gtest.h:2181:3: note: expanded from macro 'GTEST_TEST'
    GTEST_TEST_(test_case_name, test_name, \
    ^
  googletest/googletest-src/googletest/include/gtest/internal/gtest-internal.h:1228:38: note: expanded from macro 'GTEST_TEST_'
          #test_case_name, #test_name, NULL, NULL, \
                                       ^
  /usr/include/clang/5.0.0/include/stddef.h:100:18: note: expanded from macro 'NULL'
  #    define NULL __null
                   ^

Perhaps it should not complain about such system headers / headers included via `-isystem <path>`?


https://reviews.llvm.org/D32914