maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity

[clangd] Add ) to signature-help triggers 

It is important for nested function calls.

Differential Revision: https://reviews.llvm.org/D115799
This commit is contained in:
Kadir Cetinkaya 2021-12-15 14:33:34 +01:00
parent dc7b672f96
commit 517f1d9e5c
No known key found for this signature in database
GPG Key ID: E39E36B8D2057ED6
2 changed files with 3 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
clang-tools-extra/clangd/ClangdLSPServer.cpp
View File

@ -555,7 +555,7 @@ void ClangdLSPServer::onInitialize(const InitializeParams &Params,
}},
{"signatureHelpProvider",
llvm::json::Object{
{"triggerCharacters", {"(", ","}},
{"triggerCharacters", {"(", ",", ")"}},
}},
{"declarationProvider", true},
{"definitionProvider", true},

3
clang-tools-extra/clangd/test/initialize-params.test
View File

@ -107,7 +107,8 @@
# CHECK-NEXT: "signatureHelpProvider": {
# CHECK-NEXT: "triggerCharacters": [
# CHECK-NEXT: "(",
# CHECK-NEXT: ","
# CHECK-NEXT: ",",
# CHECK-NEXT: ")"
# CHECK-NEXT: ]
# CHECK-NEXT: },
# CHECK-NEXT: "textDocumentSync": {