commit ea223a7a6a99c744bc13682ddb16c245939229bc
author Daniel Jasper <djasper@google.com> Fri Aug 14 13:39:57 2015 +0000
committer Daniel Jasper <djasper@google.com> Fri Aug 14 13:39:57 2015 +0000
tree 9fea04880a5078ab945a44743d4811a6613b6b0b
parent 90eb70c8a76db399d9128a4f70af8cd393374694

misc-unused-parameters: Fix crasher with C forward declarations that
can leave out the parameter list.

llvm-svn: 245048

clang-tools-extra/clang-tidy/misc/UnusedParametersCheck.cpp

diff --git a/clang-tools-extra/clang-tidy/misc/UnusedParametersCheck.cpp b/clang-tools-extra/clang-tidy/misc/UnusedParametersCheck.cpp
index 326fb05..15c582b 100644
--- a/clang-tools-extra/clang-tidy/misc/UnusedParametersCheck.cpp
+++ b/clang-tools-extra/clang-tidy/misc/UnusedParametersCheck.cpp
@@ -84,7 +84,8 @@
 
   // Fix all redeclarations.
   for (const FunctionDecl *FD : Function->redecls())
-    MyDiag << removeParameter(FD, ParamIndex);
+    if (FD->param_size())
+      MyDiag << removeParameter(FD, ParamIndex);
 
   // Fix all call sites.
   auto CallMatches = ast_matchers::match(