Do not mandate LanguageServerApplication running

Remove an execution path that only works if IApplication is running; so JDT-LS can be used in different deployments. Part of #2311
eclipse-jdtls · Nov 9, 2022 · e7fc9a8 · e7fc9a8
1 parent 32e4780
commit e7fc9a8
Show file tree

Hide file tree

Showing 5 changed files with 14 additions and 11 deletions.
diff --git a/org.eclipse.jdt.ls.core/plugin.xml b/org.eclipse.jdt.ls.core/plugin.xml
@@ -13,7 +13,7 @@
             thread="main"
             visible="true">
          <run
-               class="org.eclipse.jdt.ls.core.internal.LanguageServer">
+               class="org.eclipse.jdt.ls.core.internal.LanguageServerApplication">
          </run>
       </application>
    </extension>

diff --git a/org.eclipse.jdt.ls.core/src/org/eclipse/jdt/ls/core/internal/JavaLanguageServerPlugin.java b/org.eclipse.jdt.ls.core/src/org/eclipse/jdt/ls/core/internal/JavaLanguageServerPlugin.java
@@ -122,7 +122,7 @@ public class JavaLanguageServerPlugin extends Plugin {
 
 	private ISourceDownloader sourceDownloader;
 
-	private LanguageServer languageServer;
+	private LanguageServerApplication languageServer;
 	private ProjectsManager projectsManager;
 	private DigestStore digestStore;
 	private ContentProviderManager contentProviderManager;
@@ -140,7 +140,7 @@ public class JavaLanguageServerPlugin extends Plugin {
 
 	private ExecutorService executorService;
 
-	public static LanguageServer getLanguageServer() {
+	public static LanguageServerApplication getLanguageServer() {
 		return pluginInstance == null ? null : pluginInstance.languageServer;
 	}
 
@@ -457,7 +457,7 @@ public static void sendStatus(ServiceStatus serverStatus, String status) {
 		}
 	}
 
-	static void startLanguageServer(LanguageServer newLanguageServer) throws IOException {
+	static void startLanguageServer(LanguageServerApplication newLanguageServer) throws IOException {
 		if (pluginInstance != null) {
 			pluginInstance.languageServer = newLanguageServer;
 			pluginInstance.startConnection();

diff --git a/.../jdt/ls/core/internal/LanguageServer.java → ...e/internal/LanguageServerApplication.java b/.../jdt/ls/core/internal/LanguageServer.java → ...e/internal/LanguageServerApplication.java
@@ -15,10 +15,12 @@
 import org.eclipse.equinox.app.IApplication;
 import org.eclipse.equinox.app.IApplicationContext;
 
-public class LanguageServer implements IApplication {
+public class LanguageServerApplication implements IApplication {
+
+	// Make it static 0 or 1 IApplication can be served at a time
+	private static long parentProcessId;
 
   private volatile boolean shutdown;
-  private long parentProcessId;
   private final Object waitLock = new Object();
 
 	@Override
@@ -54,8 +56,8 @@ public void exit() {
     }
   }
 
-	public void setParentProcessId(long pid) {
-		this.parentProcessId = pid;
+	public static void setParentProcessId(long pid) {
+		parentProcessId = pid;
 	}
 
 	/**

diff --git a/org.eclipse.jdt.ls.core/src/org/eclipse/jdt/ls/core/internal/ParentProcessWatcher.java b/org.eclipse.jdt.ls.core/src/org/eclipse/jdt/ls/core/internal/ParentProcessWatcher.java
@@ -36,11 +36,11 @@ public final class ParentProcessWatcher implements Runnable, Function<MessageCon
 	private static final boolean isJava1x = System.getProperty("java.version").startsWith("1.");
 	private static final int POLL_DELAY_SECS = 10;
 	private volatile long lastActivityTime;
-	private final LanguageServer server;
+	private final LanguageServerApplication server;
 	private ScheduledFuture<?> task;
 	private ScheduledExecutorService service;
 
-	public ParentProcessWatcher(LanguageServer server ) {
+	public ParentProcessWatcher(LanguageServerApplication server) {
 		this.server = server;
 		if (ProcessHandle.current().parent().isPresent()) {
 			this.server.setParentProcessId(ProcessHandle.current().parent().get().pid());

diff --git a/org.eclipse.jdt.ls.core/src/org/eclipse/jdt/ls/core/internal/handlers/BaseInitHandler.java b/org.eclipse.jdt.ls.core/src/org/eclipse/jdt/ls/core/internal/handlers/BaseInitHandler.java
@@ -30,6 +30,7 @@
 import org.eclipse.jdt.ls.core.internal.JSONUtility;
 import org.eclipse.jdt.ls.core.internal.JVMConfigurator;
 import org.eclipse.jdt.ls.core.internal.JavaLanguageServerPlugin;
+import org.eclipse.jdt.ls.core.internal.LanguageServerApplication;
 import org.eclipse.jdt.ls.core.internal.ResourceUtils;
 import org.eclipse.jdt.ls.core.internal.managers.ProjectsManager;
 import org.eclipse.jdt.ls.core.internal.preferences.PreferenceManager;
@@ -152,7 +153,7 @@ public InitializeResult initialize(InitializeParams param) {
 
 		Integer processId = param.getProcessId();
 		if (processId != null) {
-			JavaLanguageServerPlugin.getLanguageServer().setParentProcessId(processId.longValue());
+			LanguageServerApplication.setParentProcessId(processId.longValue());
 		}
 
 		return initializationOptions;