[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