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

GDScript: Disable LSP if either jsonrpc or websocket are disabled

Browse source 
Follow-up to #29780.
This commit is contained in:
Rémi Verschelde 2019-08-28 13:10:32 +02:00
parent 46ad60385b
commit b994124df3
2 changed files with 19 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
modules/gdscript/SCsub
View file

@ -9,4 +9,11 @@ env_gdscript.add_source_files(env.modules_sources, "*.cpp")
if env['tools']: if env['tools']:
env_gdscript.add_source_files(env.modules_sources, "./editor/*.cpp") env_gdscript.add_source_files(env.modules_sources, "./editor/*.cpp")
# Those two modules are required for the language server protocol
if env['module_jsonrpc_enabled'] and env['module_websocket_enabled']:
env_gdscript.add_source_files(env.modules_sources, "./language_server/*.cpp") env_gdscript.add_source_files(env.modules_sources, "./language_server/*.cpp")
else:
# Using a define in the disabled case, to avoid having an extra define
# in regular builds where all modules are enabled.
env_gdscript.Append(CPPDEFINES=['GDSCRIPT_NO_LSP'])

15
modules/gdscript/register_types.cpp
View file

@ -34,10 +34,8 @@
#include "core/io/resource_loader.h" #include "core/io/resource_loader.h"
#include "core/os/dir_access.h" #include "core/os/dir_access.h"
#include "core/os/file_access.h" #include "core/os/file_access.h"
#include "editor/gdscript_highlighter.h"
#include "gdscript.h" #include "gdscript.h"
#include "gdscript_tokenizer.h" #include "gdscript_tokenizer.h"
#include "language_server/gdscript_language_server.h"
GDScriptLanguage *script_language_gd = NULL; GDScriptLanguage *script_language_gd = NULL;
Ref<ResourceFormatLoaderGDScript> resource_loader_gd; Ref<ResourceFormatLoaderGDScript> resource_loader_gd;
@ -45,10 +43,15 @@ Ref<ResourceFormatSaverGDScript> resource_saver_gd;
#ifdef TOOLS_ENABLED #ifdef TOOLS_ENABLED
#include "core/engine.h"
#include "editor/editor_export.h" #include "editor/editor_export.h"
#include "editor/editor_node.h" #include "editor/editor_node.h"
#include "editor/editor_settings.h" #include "editor/editor_settings.h"
#include "editor/gdscript_highlighter.h"
#ifndef GDSCRIPT_NO_LSP
#include "core/engine.h"
#include "language_server/gdscript_language_server.h"
#endif // !GDSCRIPT_NO_LSP
class EditorExportGDScript : public EditorExportPlugin { class EditorExportGDScript : public EditorExportPlugin {
@ -137,13 +140,15 @@ static void _editor_init() {
gd_export.instance(); gd_export.instance();
EditorExport::get_singleton()->add_export_plugin(gd_export); EditorExport::get_singleton()->add_export_plugin(gd_export);
#ifndef GDSCRIPT_NO_LSP
register_lsp_types(); register_lsp_types();
GDScriptLanguageServer *lsp_plugin = memnew(GDScriptLanguageServer); GDScriptLanguageServer *lsp_plugin = memnew(GDScriptLanguageServer);
EditorNode::get_singleton()->add_editor_plugin(lsp_plugin); EditorNode::get_singleton()->add_editor_plugin(lsp_plugin);
Engine::get_singleton()->add_singleton(Engine::Singleton("GDScriptLanguageProtocol", GDScriptLanguageProtocol::get_singleton())); Engine::get_singleton()->add_singleton(Engine::Singleton("GDScriptLanguageProtocol", GDScriptLanguageProtocol::get_singleton()));
#endif // !GDSCRIPT_NO_LSP
} }
#endif #endif // TOOLS_ENABLED
void register_gdscript_types() { void register_gdscript_types() {
@ -162,7 +167,7 @@ void register_gdscript_types() {
#ifdef TOOLS_ENABLED #ifdef TOOLS_ENABLED
ScriptEditor::register_create_syntax_highlighter_function(GDScriptSyntaxHighlighter::create); ScriptEditor::register_create_syntax_highlighter_function(GDScriptSyntaxHighlighter::create);
EditorNode::add_init_callback(_editor_init); EditorNode::add_init_callback(_editor_init);
#endif #endif // TOOLS_ENABLED
} }
void unregister_gdscript_types() { void unregister_gdscript_types() {