malcolm.parsons added inline comments.
================
Comment at: clang-tidy/readability/AvoidConstParamsInDecls.cpp:39
@@ -38,2 +38,3 @@
// generate a non-definition FunctionDecl too. Ignore those.
- unless(cxxMethodDecl(ofClass(cxxRecordDecl(isLambda())))),
+ // Ignore template instantiations too.
+ unless(cxxMethodDecl(ofClass(cxxRecordDecl(anyOf(
----------------
alexfh wrote:
> Would be helpful to expand on why we're ignoring only member function of
> class instantiations (and not instantiations of member or free standing
> functions, for example).
AFAICT, template instantiations of member and free standing functions are
definitions when the template is a definition.
https://reviews.llvm.org/D24652
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
