alexfh added inline comments.
================
Comment at: clang-tidy/misc/UnusedParametersCheck.cpp:141
!Indexer->getOtherRefs(Function).empty() || isOverrideMethod(Function)) {
- SourceRange RemovalRange(Param->getLocation(),
- Param->DeclaratorDecl::getSourceRange().getEnd());
+ SourceRange RemovalRange(Param->getLocation(), Param->Decl::getLocation());
// Note: We always add a space before the '/*' to not accidentally create a
----------------
BTW, can we just remove a single token representing the parameter name? E.g.
`SourceRange RemovalRange(Param->getLocation());`.
Repository:
rL LLVM
https://reviews.llvm.org/D37846
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
