10 #include "../utils/LexerUtils.h" 11 #include "clang/AST/ASTContext.h" 12 #include "clang/ASTMatchers/ASTMatchFinder.h" 20 void SuspiciousSemicolonCheck::registerMatchers(MatchFinder *Finder) {
22 stmt(anyOf(ifStmt(hasThen(nullStmt().bind(
"semi")),
23 unless(hasElse(stmt()))),
24 forStmt(hasBody(nullStmt().bind(
"semi"))),
25 cxxForRangeStmt(hasBody(nullStmt().bind(
"semi"))),
26 whileStmt(hasBody(nullStmt().bind(
"semi")))))
31 void SuspiciousSemicolonCheck::check(
const MatchFinder::MatchResult &
Result) {
32 if (Result.Context->getDiagnostics().hasUncompilableErrorOccurred())
35 const auto *Semicolon = Result.Nodes.getNodeAs<NullStmt>(
"semi");
36 SourceLocation LocStart = Semicolon->getBeginLoc();
38 if (LocStart.isMacroID())
41 ASTContext &Ctxt = *Result.Context;
44 auto &SM = *Result.SourceManager;
45 unsigned SemicolonLine = SM.getSpellingLineNumber(LocStart);
47 const auto *Statement = Result.Nodes.getNodeAs<Stmt>(
"stmt");
48 const bool IsIfStmt = isa<IfStmt>(Statement);
51 SM.getSpellingLineNumber(Token.getLocation()) != SemicolonLine)
54 SourceLocation LocEnd = Semicolon->getEndLoc();
55 FileID FID = SM.getFileID(LocEnd);
56 const llvm::MemoryBuffer *Buffer = SM.getBuffer(FID, LocEnd);
57 Lexer Lexer(SM.getLocForStartOfFile(FID), Ctxt.getLangOpts(),
58 Buffer->getBufferStart(), SM.getCharacterData(LocEnd) + 1,
59 Buffer->getBufferEnd());
60 if (Lexer.LexFromRawLexer(Token))
63 unsigned BaseIndent = SM.getSpellingColumnNumber(Statement->getBeginLoc());
64 unsigned NewTokenIndent = SM.getSpellingColumnNumber(Token.getLocation());
65 unsigned NewTokenLine = SM.getSpellingLineNumber(Token.getLocation());
67 if (!IsIfStmt && NewTokenIndent <= BaseIndent &&
68 Token.getKind() != tok::l_brace && NewTokenLine != SemicolonLine)
71 diag(LocStart,
"potentially unintended semicolon")
72 << FixItHint::CreateRemoval(SourceRange(LocStart, LocEnd));
Token getPreviousToken(SourceLocation Location, const SourceManager &SM, const LangOptions &LangOpts, bool SkipComments)
Returns previous token or tok::unknown if not found.
===– Representation.cpp - ClangDoc Representation --------—*- C++ -*-===//
llvm::Optional< llvm::Expected< tooling::AtomicChanges > > Result