diff options
| author | Daniel Jasper <djasper@google.com> | 2015-08-10 15:45:46 +0000 |
|---|---|---|
| committer | Daniel Jasper <djasper@google.com> | 2015-08-10 15:45:46 +0000 |
| commit | b3a74c659b8946917a7b658abb1ae8b0d00ba70f (patch) | |
| tree | cffa7bff9264b51a07f06c50a08635e84c12dec7 /clang-tools-extra/clang-tidy/misc/UnusedParametersCheck.cpp | |
| parent | 57add8ddfb91bead2b2e4b0a1263d39ba4ad08d0 (diff) | |
| download | bcm5719-llvm-b3a74c659b8946917a7b658abb1ae8b0d00ba70f.tar.gz bcm5719-llvm-b3a74c659b8946917a7b658abb1ae8b0d00ba70f.zip | |
misc-unused-parameters: Don't touch K&R style functions.
We couldn't calculate the removal ranges properly at this point.
llvm-svn: 244454
Diffstat (limited to 'clang-tools-extra/clang-tidy/misc/UnusedParametersCheck.cpp')
| -rw-r--r-- | clang-tools-extra/clang-tidy/misc/UnusedParametersCheck.cpp | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/clang-tools-extra/clang-tidy/misc/UnusedParametersCheck.cpp b/clang-tools-extra/clang-tidy/misc/UnusedParametersCheck.cpp index f7770cc8e40..326fb0595e0 100644 --- a/clang-tools-extra/clang-tidy/misc/UnusedParametersCheck.cpp +++ b/clang-tools-extra/clang-tidy/misc/UnusedParametersCheck.cpp @@ -97,7 +97,8 @@ void UnusedParametersCheck::warnOnUnusedParameter( void UnusedParametersCheck::check(const MatchFinder::MatchResult &Result) { const auto *Function = Result.Nodes.getNodeAs<FunctionDecl>("function"); - if (!Function->doesThisDeclarationHaveABody()) + if (!Function->doesThisDeclarationHaveABody() || + !Function->hasWrittenPrototype()) return; for (unsigned i = 0, e = Function->getNumParams(); i != e; ++i) { const auto *Param = Function->getParamDecl(i); |
