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

Haojian Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 6 08:34:35 PST 2017


hokein added inline comments.


================
Comment at: clang-tools-extra/trunk/test/clang-tidy/modernize-use-nullptr.cpp:254
+
+  void h(T *default_value = 0) {}
+
----------------
alexfh wrote:
> 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.
+1, patch are welcome.


Repository:
  rL LLVM

https://reviews.llvm.org/D30639





More information about the cfe-commits mailing list