Use simple name length instead of javac node length.

eclipse-jdtls · Jul 16, 2024 · 0ce7ac2 · 0ce7ac2
1 parent 6df7311
commit 0ce7ac2
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/org.eclipse.jdt.core.javac/src/org/eclipse/jdt/core/dom/JavacConverter.java b/org.eclipse.jdt.core.javac/src/org/eclipse/jdt/core/dom/JavacConverter.java
@@ -393,7 +393,7 @@ private void nameSettings(SimpleName name, JCMethodDecl javac) {
 		if (javac.getReturnType() != null) {
 			start = javac.getReturnType().getEndPosition(this.javacCompilationUnit.endPositions);
 		}
-		var length = javac.getName().length();
+		var length = name.getLength(); // use SimpleName since it can change due to erroneous scenarios
 		name.setSourceRange(start, length);
 	}
 
@@ -402,7 +402,7 @@ private void nameSettings(SimpleName name, JCVariableDecl javac) {
 		if (javac.getType() != null) {
 			start = javac.getType().getEndPosition(this.javacCompilationUnit.endPositions);
 		}
-		var length = javac.getName().length();
+		var length = name.getLength(); // use SimpleName since it can change due to erroneous scenarios
 		name.setSourceRange(start, length);
 	}