[PATCH] D30639: [clang-tidy] Ignore substituted template types in modernize-use-nullptr check.

Alexander Kornienko via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 6 08:21:56 PST 2017


alexfh added inline comments.


================
Comment at: clang-tools-extra/trunk/test/clang-tidy/modernize-use-nullptr.cpp:254
+
+  void h(T *default_value = 0) {}
+
----------------
xazax.hun wrote:
> Maybe as a separate patch, but I think it might be worth to warn here. WDYT? (Sorry for the double post, the previous one is disappeared after the commit.)
Seems reasonable, but probably not an extremely frequent case. Deserves a FIXME at least.


Repository:
  rL LLVM

https://reviews.llvm.org/D30639





More information about the cfe-commits mailing list