[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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits