Skip to content

Commit 43f616a

Browse files
RKSimonmemfrob
authored andcommitted
[clangd] Fix MSVC implicit capture build failure.
MSVC builds were failing because the constexpr wasn't couldn't be captured by the lamdba. Fix an implicit double to float truncation warning as well.
1 parent f9b85f3 commit 43f616a

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

clang-tools-extra/clangd/unittests/CodeCompleteTests.cpp

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -196,10 +196,10 @@ TEST(DecisionForestRankingModel, ReferencesAffectRanking) {
196196

197197
TEST(DecisionForestRankingModel, DecisionForestScorerCallbackTest) {
198198
clangd::CodeCompleteOptions Opts;
199-
constexpr float MagicNumber = 1234.5678;
199+
constexpr float MagicNumber = 1234.5678f;
200200
Opts.RankingModel = CodeCompleteOptions::DecisionForest;
201-
Opts.DecisionForestScorer = [](const SymbolQualitySignals &,
202-
const SymbolRelevanceSignals &, float Base) {
201+
Opts.DecisionForestScorer = [&](const SymbolQualitySignals &,
202+
const SymbolRelevanceSignals &, float Base) {
203203
DecisionForestScores Scores;
204204
Scores.Total = MagicNumber;
205205
Scores.ExcludingName = MagicNumber;

0 commit comments

Comments
 (0)