[clang-tools-extra] [clang-tools-extra] Remove redundant calls to std::unique_ptr<T>::get (NFC) (PR #138774)
Kazu Hirata via cfe-commits
cfe-commits at lists.llvm.org
Tue May 6 15:34:38 PDT 2025
https://github.com/kazutakahirata created https://github.com/llvm/llvm-project/pull/138774
None
>From a524a8098d5a513259d0d82b262d4b5a769d4d1f Mon Sep 17 00:00:00 2001
From: Kazu Hirata <kazu at google.com>
Date: Tue, 6 May 2025 15:20:38 -0700
Subject: [PATCH] [clang-tools-extra] Remove redundant calls to
std::unique_ptr<T>::get (NFC)
---
.../clang-change-namespace/ChangeNamespace.cpp | 7 +++----
.../clangd/unittests/PrerequisiteModulesTest.cpp | 4 ++--
2 files changed, 5 insertions(+), 6 deletions(-)
diff --git a/clang-tools-extra/clang-change-namespace/ChangeNamespace.cpp b/clang-tools-extra/clang-change-namespace/ChangeNamespace.cpp
index 850df7daf5c03..3e367ab1a5558 100644
--- a/clang-tools-extra/clang-change-namespace/ChangeNamespace.cpp
+++ b/clang-tools-extra/clang-change-namespace/ChangeNamespace.cpp
@@ -113,7 +113,7 @@ static SourceLocation getStartOfNextLine(SourceLocation Loc,
const SourceManager &SM,
const LangOptions &LangOpts) {
std::unique_ptr<Lexer> Lex = getLexerStartingFromLoc(Loc, SM, LangOpts);
- if (!Lex.get())
+ if (!Lex)
return SourceLocation();
llvm::SmallVector<char, 16> Line;
// FIXME: this is a bit hacky to get ReadToEndOfLine work.
@@ -647,9 +647,8 @@ static SourceLocation getLocAfterNamespaceLBrace(const NamespaceDecl *NsDecl,
const LangOptions &LangOpts) {
std::unique_ptr<Lexer> Lex =
getLexerStartingFromLoc(NsDecl->getBeginLoc(), SM, LangOpts);
- assert(Lex.get() &&
- "Failed to create lexer from the beginning of namespace.");
- if (!Lex.get())
+ assert(Lex && "Failed to create lexer from the beginning of namespace.");
+ if (!Lex)
return SourceLocation();
Token Tok;
while (!Lex->LexFromRawLexer(Tok) && Tok.isNot(tok::TokenKind::l_brace)) {
diff --git a/clang-tools-extra/clangd/unittests/PrerequisiteModulesTest.cpp b/clang-tools-extra/clangd/unittests/PrerequisiteModulesTest.cpp
index 27f4c817a8ff3..03e65768c26a6 100644
--- a/clang-tools-extra/clangd/unittests/PrerequisiteModulesTest.cpp
+++ b/clang-tools-extra/clangd/unittests/PrerequisiteModulesTest.cpp
@@ -545,8 +545,8 @@ void func() {
EXPECT_TRUE(Preamble);
EXPECT_TRUE(Preamble->RequiredModules);
- auto Result = signatureHelp(getFullPath("Use.cpp"), Test.point(),
- *Preamble.get(), Use, MarkupKind::PlainText);
+ auto Result = signatureHelp(getFullPath("Use.cpp"), Test.point(), *Preamble,
+ Use, MarkupKind::PlainText);
EXPECT_FALSE(Result.signatures.empty());
EXPECT_EQ(Result.signatures[0].label, "printA(int a) -> void");
EXPECT_EQ(Result.signatures[0].parameters[0].labelString, "int a");
More information about the cfe-commits
mailing list