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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
