[clang-tools-extra] [clang-tidy] Diagnose NULL macro expansions in implicit-bool-conversion (PR #188694)

[Zeyi Xu via cfe-commits]

================
@@ -396,11 +396,14 @@ Changes in existing checks
   it easier to see which specific enumerators need explicit initialization.
 
 - Improved :doc:`readability-implicit-bool-conversion
-  <clang-tidy/checks/readability/implicit-bool-conversion>` check by fixing a
-  false positive where `AllowPointerConditions` and `AllowIntegerConditions`
-  options did not suppress warnings when the condition expression involved
-  temporaries (e.g. passing a string literal to a ``const std::string&``
-  parameter)
+  <clang-tidy/checks/readability/implicit-bool-conversion>` check:
+
+  - Fixed a false positive where `AllowPointerConditions` and
+    `AllowIntegerConditions` options did not suppress warnings when the
+    condition expression involved temporaries (e.g. passing a string literal to
+    a ``const std::string&`` parameter).
+
+  - Warn and provide fix-its when ``NULL`` is defined in system headers.
----------------
zeyi2 wrote:

fixed, thanks!

https://github.com/llvm/llvm-project/pull/188694