[clang-tools-extra] [clang-tidy] Fix warnings caused by "new check" template (PR #80537)
Danny Mösch via cfe-commits
cfe-commits at lists.llvm.org
Sat Feb 3 02:33:25 PST 2024
https://github.com/SimplyDanny created https://github.com/llvm/llvm-project/pull/80537
None
>From 3ba9198670311c38fa145ef0af57edd60120568a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Danny=20M=C3=B6sch?= <danny.moesch at icloud.com>
Date: Sat, 3 Feb 2024 11:31:51 +0100
Subject: [PATCH 1/2] [clang-tidy] Replace deprecated method in template
---
clang-tools-extra/clang-tidy/add_new_check.py | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/clang-tools-extra/clang-tidy/add_new_check.py b/clang-tools-extra/clang-tidy/add_new_check.py
index ada2ee1119cf9..eecff6082f3d6 100755
--- a/clang-tools-extra/clang-tidy/add_new_check.py
+++ b/clang-tools-extra/clang-tidy/add_new_check.py
@@ -146,7 +146,7 @@ def write_implementation(module_path, module, namespace, check_name_camel):
void %(check_name)s::check(const MatchFinder::MatchResult &Result) {
// FIXME: Add callback implementation.
const auto *MatchedDecl = Result.Nodes.getNodeAs<FunctionDecl>("x");
- if (!MatchedDecl->getIdentifier() || MatchedDecl->getName().startswith("awesome_"))
+ if (!MatchedDecl->getIdentifier() || MatchedDecl->getName().starts_with("awesome_"))
return;
diag(MatchedDecl->getLocation(), "function %%0 is insufficiently awesome")
<< MatchedDecl
>From 4b487beaef9965973c82092600eade9c57f00448 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Danny=20M=C3=B6sch?= <danny.moesch at icloud.com>
Date: Sat, 3 Feb 2024 11:32:09 +0100
Subject: [PATCH 2/2] [clang-tidy] Omit (yet) unused include
---
clang-tools-extra/clang-tidy/add_new_check.py | 1 -
1 file changed, 1 deletion(-)
diff --git a/clang-tools-extra/clang-tidy/add_new_check.py b/clang-tools-extra/clang-tidy/add_new_check.py
index eecff6082f3d6..a6af76809af02 100755
--- a/clang-tools-extra/clang-tidy/add_new_check.py
+++ b/clang-tools-extra/clang-tidy/add_new_check.py
@@ -131,7 +131,6 @@ def write_implementation(module_path, module, namespace, check_name_camel):
//===----------------------------------------------------------------------===//
#include "%(check_name)s.h"
-#include "clang/AST/ASTContext.h"
#include "clang/ASTMatchers/ASTMatchFinder.h"
using namespace clang::ast_matchers;
More information about the cfe-commits
mailing list