diff options
author | thegeorg <thegeorg@yandex-team.com> | 2024-03-13 13:58:24 +0300 |
---|---|---|
committer | thegeorg <thegeorg@yandex-team.com> | 2024-03-13 14:11:53 +0300 |
commit | 11a895b7e15d1c5a1f52706396b82e3f9db953cb (patch) | |
tree | fabc6d883b0f946151f61ae7865cee9f529a1fdd /contrib/libs/clang16/tools/extra/clang-tidy/bugprone/TerminatingContinueCheck.cpp | |
parent | 9685917341315774aad5733b1793b1e533a88bbb (diff) | |
download | ydb-11a895b7e15d1c5a1f52706396b82e3f9db953cb.tar.gz |
Export clang-format16 via ydblib project
6e6be3a95868fde888d801b7590af4044049563f
Diffstat (limited to 'contrib/libs/clang16/tools/extra/clang-tidy/bugprone/TerminatingContinueCheck.cpp')
-rw-r--r-- | contrib/libs/clang16/tools/extra/clang-tidy/bugprone/TerminatingContinueCheck.cpp | 45 |
1 files changed, 45 insertions, 0 deletions
diff --git a/contrib/libs/clang16/tools/extra/clang-tidy/bugprone/TerminatingContinueCheck.cpp b/contrib/libs/clang16/tools/extra/clang-tidy/bugprone/TerminatingContinueCheck.cpp new file mode 100644 index 0000000000..e8f63fb709 --- /dev/null +++ b/contrib/libs/clang16/tools/extra/clang-tidy/bugprone/TerminatingContinueCheck.cpp @@ -0,0 +1,45 @@ +//===--- TerminatingContinueCheck.cpp - clang-tidy-------------------------===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception +// +//===----------------------------------------------------------------------===// + +#include "TerminatingContinueCheck.h" +#include "clang/AST/ASTContext.h" +#include "clang/ASTMatchers/ASTMatchFinder.h" +#include "clang/Lex/Lexer.h" +#include "clang/Tooling/FixIt.h" + +using namespace clang::ast_matchers; + +namespace clang::tidy::bugprone { + +void TerminatingContinueCheck::registerMatchers(MatchFinder *Finder) { + const auto DoWithFalse = + doStmt(hasCondition(ignoringImpCasts( + anyOf(cxxBoolLiteral(equals(false)), integerLiteral(equals(0)), + cxxNullPtrLiteralExpr(), gnuNullExpr()))), + equalsBoundNode("closestLoop")); + + Finder->addMatcher( + continueStmt( + hasAncestor(stmt(anyOf(forStmt(), whileStmt(), cxxForRangeStmt(), + doStmt(), switchStmt())) + .bind("closestLoop")), + hasAncestor(DoWithFalse)) + .bind("continue"), + this); +} + +void TerminatingContinueCheck::check(const MatchFinder::MatchResult &Result) { + const auto *ContStmt = Result.Nodes.getNodeAs<ContinueStmt>("continue"); + + auto Diag = + diag(ContStmt->getBeginLoc(), + "'continue' in loop with false condition is equivalent to 'break'") + << tooling::fixit::createReplacement(*ContStmt, "break"); +} + +} // namespace clang::tidy::bugprone |