hokein added inline comments.
================
Comment at: clang-tools-extra/clangd/unittests/CodeCompleteTests.cpp:651
+ auto Results =
+ completions("int main() { abs^ }", {func("absl"), func("absb")});
EXPECT_THAT(Results.Completions,
----------------
sorry, I didn't infer the motivation of this change func->ns from this patch,
could you explain?
================
Comment at: clang-tools-extra/clangd/unittests/CodeCompleteTests.cpp:655
Results = completions("int main() { abs^ }",
- {withReferences(10000, ns("absl")), func("absb")});
- EXPECT_THAT(Results.Completions,
- HasSubsequence(Named("absl"), Named("absb")));
+ {withReferences(100, func("absl")), func("absb")});
}
----------------
looks like the Results here is not verified, as you remove the line below, is
it intentional?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D96353/new/
https://reviews.llvm.org/D96353
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
