From 8ada96e7b8f1f3778288f1c9b1bdf9b72ddcecac Mon Sep 17 00:00:00 2001 From: Kazu Hirata Date: Fri, 4 Oct 2024 10:00:28 -0700 Subject: [PATCH] [clangd] Simplify ternary expressions with std::optional::value_or (NFC) --- clang-tools-extra/clangd/FindSymbols.cpp | 2 +- clang-tools-extra/clangd/index/MemIndex.cpp | 2 +- clang-tools-extra/clangd/index/dex/Dex.cpp | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/clang-tools-extra/clangd/FindSymbols.cpp b/clang-tools-extra/clangd/FindSymbols.cpp index cf2f8b62a2841..84bcbc1f2ddd3 100644 --- a/clang-tools-extra/clangd/FindSymbols.cpp +++ b/clang-tools-extra/clangd/FindSymbols.cpp @@ -111,7 +111,7 @@ getWorkspaceSymbols(llvm::StringRef Query, int Limit, *Req.Limit *= 5; } TopN Top( - Req.Limit ? *Req.Limit : std::numeric_limits::max()); + Req.Limit.value_or(std::numeric_limits::max())); FuzzyMatcher Filter(Req.Query); Index->fuzzyFind(Req, [HintPath, &Top, &Filter, AnyScope = Req.AnyScope, diff --git a/clang-tools-extra/clangd/index/MemIndex.cpp b/clang-tools-extra/clangd/index/MemIndex.cpp index fe0ee873018b3..2665d46b97d83 100644 --- a/clang-tools-extra/clangd/index/MemIndex.cpp +++ b/clang-tools-extra/clangd/index/MemIndex.cpp @@ -31,7 +31,7 @@ bool MemIndex::fuzzyFind( trace::Span Tracer("MemIndex fuzzyFind"); TopN> Top( - Req.Limit ? *Req.Limit : std::numeric_limits::max()); + Req.Limit.value_or(std::numeric_limits::max())); FuzzyMatcher Filter(Req.Query); bool More = false; for (const auto &Pair : Index) { diff --git a/clang-tools-extra/clangd/index/dex/Dex.cpp b/clang-tools-extra/clangd/index/dex/Dex.cpp index 19dc3080f9f89..b7d3063e19b49 100644 --- a/clang-tools-extra/clangd/index/dex/Dex.cpp +++ b/clang-tools-extra/clangd/index/dex/Dex.cpp @@ -264,7 +264,7 @@ bool Dex::fuzzyFind(const FuzzyFindRequest &Req, return LHS.second > RHS.second; }; TopN Top( - Req.Limit ? *Req.Limit : std::numeric_limits::max(), Compare); + Req.Limit.value_or(std::numeric_limits::max()), Compare); for (const auto &IDAndScore : IDAndScores) { const DocID SymbolDocID = IDAndScore.first; const auto *Sym = Symbols[SymbolDocID];