aboutsummaryrefslogtreecommitdiff
path: root/clang-tools-extra/clang-tidy/modernize/UseOverrideCheck.cpp
diff options
context:
space:
mode:
authorKevin Joseph <kevinjoseph1995@gmail.com>2024-02-14 11:59:21 -0800
committerGitHub <noreply@github.com>2024-02-14 20:59:21 +0100
commit5992b3272b29e071f6f5a4807a4e0c23e88c310d (patch)
treeb193478980cc735938cfafdae91bf0a172744411 /clang-tools-extra/clang-tidy/modernize/UseOverrideCheck.cpp
parent1301bc46aea14297478bd13bcacff429e2a18c04 (diff)
downloadllvm-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.cpp12
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;
+ }
}
}
}