From fd3a3b3f291cb93d6779a0c0670a43d97579ab99 Mon Sep 17 00:00:00 2001 From: Aaron Ballman Date: Wed, 7 Oct 2015 15:14:10 +0000 Subject: Loosening the restriction on variadic function definitions so that extern "C" function definitions are permissible. llvm-svn: 249555 --- clang-tools-extra/clang-tidy/cert/VariadicFunctionDefCheck.cpp | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) (limited to 'clang-tools-extra/clang-tidy/cert/VariadicFunctionDefCheck.cpp') diff --git a/clang-tools-extra/clang-tidy/cert/VariadicFunctionDefCheck.cpp b/clang-tools-extra/clang-tidy/cert/VariadicFunctionDefCheck.cpp index ac788a3aee5..a5f8fa7e733 100644 --- a/clang-tools-extra/clang-tidy/cert/VariadicFunctionDefCheck.cpp +++ b/clang-tools-extra/clang-tidy/cert/VariadicFunctionDefCheck.cpp @@ -20,9 +20,12 @@ void VariadicFunctionDefCheck::registerMatchers(MatchFinder *Finder) { if (!getLangOpts().CPlusPlus) return; - // We only care about function *definitions* that are variadic. - Finder->addMatcher(functionDecl(isDefinition(), isVariadic()).bind("func"), - this); + // We only care about function *definitions* that are variadic, and do not + // have extern "C" language linkage. + Finder->addMatcher( + functionDecl(isDefinition(), isVariadic(), unless(isExternC())) + .bind("func"), + this); } void VariadicFunctionDefCheck::check(const MatchFinder::MatchResult &Result) { -- cgit v1.2.3