fixup! [clang-tidy] Added check to detect redundant inline keyword

Disable inline variable check for pre c++-17 code

Author: felix642

clang-tools-extra/clang-tidy/readability/RedundantInlineSpecifierCheck.cpp

@@ -61,18 +61,18 @@ void RedundantInlineSpecifierCheck::registerMatchers(MatchFinder *Finder) {
                          allOf(isDefinition(), hasAncestor(recordDecl()))))
           .bind("fun_decl"),
       this);
-
-  Finder->addMatcher(
-      varDecl(isInlineSpecified(),
-              anyOf(isInAnonymousNamespace(),
-                    allOf(isConstexpr(), hasAncestor(recordDecl()))))
-          .bind("var_decl"),
-      this);
-
   Finder->addMatcher(
       functionTemplateDecl(has(functionDecl(isInlineSpecified())))
           .bind("templ_decl"),
       this);
+  if (getLangOpts().CPlusPlus17) {
+    Finder->addMatcher(
+        varDecl(isInlineSpecified(),
+                anyOf(isInAnonymousNamespace(),
+                      allOf(isConstexpr(), hasAncestor(recordDecl()))))
+            .bind("var_decl"),
+        this);
+  }
 }
 
 template <typename T>

clang-tools-extra/test/clang-tidy/checkers/readability/redundant-inline-specifier.cpp

@@ -1,4 +1,4 @@
-// RUN: %check_clang_tidy %s readability-redundant-inline-specifier %t
+// RUN: %check_clang_tidy %s readability-redundant-inline-specifier -std=c++17 %t
 
 template <typename T> inline T f()
 // CHECK-MESSAGES: :[[@LINE-1]]:23: warning: function 'f' has inline specifier but is implicitly inlined [readability-redundant-inline-specifier]