[clang-tidy] Some improvements in 'misc-definitions-in-headers' check.

[hokein]

Remove unrelevant file.

[alexfh]

LG

clang-tidy/misc/DefinitionsInHeadersCheck.cpp

 
 void DefinitionsInHeadersCheck::storeOptions(
     ClangTidyOptions::OptionMap &Opts) {
   Options.store(Opts, "UseHeaderFileExtension", UseHeaderFileExtension);
   Options.store(Opts, "HeaderFileExtensions", RawStringHeaderFileExtensions);
 }
 
 void DefinitionsInHeadersCheck::registerMatchers(MatchFinder *Finder) {
+  if (!getLangOpts().CPlusPlus)
+    return;
   if (UseHeaderFileExtension) {
     Finder->addMatcher(
         namedDecl(anyOf(functionDecl(isDefinition()), varDecl(isDefinition())),
                   usesHeaderFileExtension(HeaderFileExtensions))
             .bind("name-decl"),
         this);
   } else {
     Finder->addMatcher(
void DefinitionsInHeadersCheck::check(const MatchFinder::MatchResult &Result) {
   // inline function with external linkage, class template, non-static function
   // template, static data member of a class template, member function of a
   // class template, or template specialization for which some template
   // parameters are not specifiedin a program provided that each definition
   // appears in a different translation unit, and provided the definitions
   // satisfy the following requirements.
   const auto *ND = Result.Nodes.getNodeAs<NamedDecl>("name-decl");
   assert(ND);
+  if (ND->isInvalidDecl())
+    return;
 
   // Internal linkage variable definitions are ignored for now:
   //   const int a = 1;
   //   static int b = 1;
   //
   // Although these might also cause ODR violations, we can be less certain and
   // should try to keep the false-positive rate down.
   if (ND->getLinkageInternal() == InternalLinkage)