From 51764b880856460e37f109b4a3a38557437014df Mon Sep 17 00:00:00 2001 From: Jesse Glick Date: Mon, 21 Nov 2022 11:10:42 -0500 Subject: [PATCH] Fix `NullPointerException` in `MavenSelectionHintsTask` --- .../modules/maven/hints/pom/MavenSelectionHintsTask.java | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/java/maven.hints/src/org/netbeans/modules/maven/hints/pom/MavenSelectionHintsTask.java b/java/maven.hints/src/org/netbeans/modules/maven/hints/pom/MavenSelectionHintsTask.java index db703f9aba83..1e36b9219e3b 100644 --- a/java/maven.hints/src/org/netbeans/modules/maven/hints/pom/MavenSelectionHintsTask.java +++ b/java/maven.hints/src/org/netbeans/modules/maven/hints/pom/MavenSelectionHintsTask.java @@ -24,6 +24,7 @@ import javax.swing.text.Document; import javax.swing.text.Position; import javax.swing.text.StyledDocument; +import org.netbeans.api.annotations.common.NonNull; import org.netbeans.api.project.FileOwnerQuery; import org.netbeans.api.project.Project; import org.netbeans.modules.maven.hints.pom.spi.SelectionPOMFixProvider; @@ -68,17 +69,18 @@ public void run(MavenResult result, SchedulerEvent event) { List errors = computeErrors(result, ss, se, cursorEvent.getCaretOffset()); HintsController.setErrors(result.getPomFile(), PomModelUtils.LAYER_POM_SELECTION, errors); } - + + @NonNull static List computeErrors(MavenResult result, int ss, int se, int co) { + final List errors = new ArrayList<>(); FileObject fo = result.getPomFile(); Project project = FileOwnerQuery.getOwner(fo); Document document = result.getSnapshot().getSource().getDocument(false); if (fo == null || project == null || project.getProjectDirectory() != fo.getParent()) { // ?? pom file ought to form a project! - return null; + return errors; } final POMModel model = result.getProjectModel(); - final List errors = new ArrayList(); // clear selection hints in case of an error; validation errors are handled by // MavenFileHintsTask. StyledDocument styled = null;