Fix out of bounds (#246)

`getEndPosition` can sometimes return `NOPOS` (`-1L`)
georgewfraser · Jul 6, 2023 · 4b17be5 · 4b17be5
1 parent da4417c
commit 4b17be5
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/main/java/org/javacs/markup/ErrorProvider.java b/src/main/java/org/javacs/markup/ErrorProvider.java
@@ -8,6 +8,7 @@
 import java.util.List;
 import java.util.regex.Pattern;
 import javax.lang.model.element.Element;
+import javax.tools.Diagnostic;
 import javax.tools.JavaFileObject;
 import org.javacs.CompileTask;
 import org.javacs.FileStore;
@@ -142,7 +143,7 @@ private org.javacs.lsp.Diagnostic warnUnused(Element unusedEl) {
         if (name.contentEquals("<init>")) {
             name = unusedEl.getEnclosingElement().getSimpleName();
         }
-        var region = contents.subSequence(start, end);
+        var region = contents.subSequence(start, end == Diagnostic.NOPOS ? contents.length() : end);
         var matcher = Pattern.compile("\\b" + name + "\\b").matcher(region);
         if (matcher.find()) {
             start += matcher.start();