diff options
author | Kevin Joseph <kevinjoseph1995@gmail.com> | 2024-02-14 11:59:21 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-02-14 20:59:21 +0100 |
commit | 5992b3272b29e071f6f5a4807a4e0c23e88c310d (patch) | |
tree | b193478980cc735938cfafdae91bf0a172744411 /clang-tools-extra/clang-tidy/modernize/UseOverrideCheck.cpp | |
parent | 1301bc46aea14297478bd13bcacff429e2a18c04 (diff) | |
download | llvm-5992b3272b29e071f6f5a4807a4e0c23e88c310d.zip llvm-5992b3272b29e071f6f5a4807a4e0c23e88c310d.tar.gz llvm-5992b3272b29e071f6f5a4807a4e0c23e88c310d.tar.bz2 |
[clangd] Clean formatting modernize-use-override (#81435)
When applying the recommended fix for the
"modernize-use-override" clang-tidy diagnostic
there was a stray whitespace. This PR fixes that.
Resolves https://github.com/clangd/clangd/issues/1704
Diffstat (limited to 'clang-tools-extra/clang-tidy/modernize/UseOverrideCheck.cpp')
-rw-r--r-- | clang-tools-extra/clang-tidy/modernize/UseOverrideCheck.cpp | 12 |
1 files changed, 9 insertions, 3 deletions
diff --git a/clang-tools-extra/clang-tidy/modernize/UseOverrideCheck.cpp b/clang-tools-extra/clang-tidy/modernize/UseOverrideCheck.cpp index e348968..fd5bd9f 100644 --- a/clang-tools-extra/clang-tidy/modernize/UseOverrideCheck.cpp +++ b/clang-tools-extra/clang-tidy/modernize/UseOverrideCheck.cpp @@ -7,6 +7,7 @@ //===----------------------------------------------------------------------===// #include "UseOverrideCheck.h" +#include "../utils/LexerUtils.h" #include "clang/AST/ASTContext.h" #include "clang/ASTMatchers/ASTMatchFinder.h" #include "clang/Lex/Lexer.h" @@ -228,9 +229,14 @@ void UseOverrideCheck::check(const MatchFinder::MatchResult &Result) { if (HasVirtual) { for (Token Tok : Tokens) { if (Tok.is(tok::kw_virtual)) { - Diag << FixItHint::CreateRemoval(CharSourceRange::getTokenRange( - Tok.getLocation(), Tok.getLocation())); - break; + std::optional<Token> NextToken = + utils::lexer::findNextTokenIncludingComments( + Tok.getEndLoc(), Sources, getLangOpts()); + if (NextToken.has_value()) { + Diag << FixItHint::CreateRemoval(CharSourceRange::getCharRange( + Tok.getLocation(), NextToken->getLocation())); + break; + } } } } |