[Debug] Abort adapter if variable cannot be found

This prevents some NPE
eclipse-lsp4e · Feb 8, 2022 · f0f243d · f0f243d
1 parent fee42d2
commit f0f243d
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/org.eclipse.lsp4e.debug/META-INF/MANIFEST.MF b/org.eclipse.lsp4e.debug/META-INF/MANIFEST.MF
@@ -2,7 +2,7 @@ Manifest-Version: 1.0
 Bundle-ManifestVersion: 2
 Bundle-Name: Debug Adapter client for Eclipse IDE (Incubation)
 Bundle-SymbolicName: org.eclipse.lsp4e.debug;singleton:=true
-Bundle-Version: 0.13.2.qualifier
+Bundle-Version: 0.13.3.qualifier
 Bundle-Activator: org.eclipse.lsp4e.debug.DSPPlugin
 Require-Bundle: org.eclipse.ui,
  org.eclipse.core.runtime,

diff --git a/...eclipse.lsp4e.debug/src/org/eclipse/lsp4e/debug/breakpoints/TextSelectionToIVariable.java b/...eclipse.lsp4e.debug/src/org/eclipse/lsp4e/debug/breakpoints/TextSelectionToIVariable.java
@@ -58,6 +58,9 @@ private IVariable getVariableFor(TextSelection selection) {
 			if (variableName.isEmpty()) {
 				variableName = findVariableName(document, selection.getOffset());
 			}
+			if (variableName == null || variableName.isEmpty()) {
+				return null;
+			}
 
 			if (Boolean.TRUE.equals(frame.getDebugTarget().getCapabilities().getSupportsEvaluateForHovers())) {
 				try {