asgardius/virtualx-engine
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

Merge pull request #39556 from akien-mga/lsp-fix-39548

Browse source 
GDScript LSP: Fix wrong error checks added in #39385
This commit is contained in:
Rémi Verschelde 2020-06-15 14:27:41 +02:00 committed by GitHub
commit e720118a75
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
2 changed files with 3 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
modules/gdscript/language_server/gdscript_language_protocol.cpp
View file

@ -190,9 +190,7 @@ Dictionary GDScriptLanguageProtocol::initialize(const Dictionary &p_params) {
params["path"] = workspace->root;
Dictionary request = make_notification("gdscript_client/changeWorkspace", params);
ERR_FAIL_COND_V_MSG(latest_client_id == -1, ret.to_json(),
"GDScriptLanguageProtocol: Can't initialize as no client is connected.");
ERR_FAIL_INDEX_V_MSG((uint64_t)latest_client_id, clients.size(), ret.to_json(),
ERR_FAIL_COND_V_MSG(!clients.has(latest_client_id), ret.to_json(),
vformat("GDScriptLanguageProtocol: Can't initialize invalid peer '%d'.", latest_client_id));
Ref<LSPeer> peer = clients.get(latest_client_id);
if (peer != nullptr) {
@ -277,7 +275,7 @@ void GDScriptLanguageProtocol::notify_client(const String &p_method, const Varia
"GDScript LSP: Can't notify client as none was connected.");
p_client_id = latest_client_id;
}
ERR_FAIL_INDEX((uint64_t)p_client_id, clients.size());
ERR_FAIL_COND(!clients.has(p_client_id));
Ref<LSPeer> peer = clients.get(p_client_id);
ERR_FAIL_COND(peer == nullptr);

2
modules/gdscript/language_server/gdscript_language_protocol.h
View file

@ -70,7 +70,7 @@ private:
HashMap<int, Ref<LSPeer>> clients;
Ref<TCP_Server> server;
int latest_client_id = -1;
int latest_client_id = 0;
int next_client_id = 0;
Ref<GDScriptTextDocument> text_document;