| Summary: | NPE in org.eclipse.xtext.util.Strings.pack | ||
|---|---|---|---|
| Product: | [Modeling] TMF | Reporter: | Michael Clay <clay> |
| Component: | Xtext | Assignee: | Project Inbox <tmf.xtext-inbox> |
| Status: | CLOSED FIXED | QA Contact: | |
| Severity: | normal | ||
| Priority: | P3 | CC: | sebastian.zarnekow, sven.efftinge |
| Version: | 1.0.1 | Flags: | sebastian.zarnekow:
indigo+
|
| Target Milestone: | M2 | ||
| Hardware: | PC | ||
| OS: | Mac OS X - Carbon (unsup.) | ||
| Whiteboard: | |||
In which cases does the array contain null entries? Maybe we should make sure they are not contained in the first place and make it explicit (through JavaDoc) that null values are not allowed in Strings.pack(). (In reply to comment #1) > In which cases does the array contain null entries? > Maybe we should make sure they are not contained in the first place and make it > explicit (through JavaDoc) that null values are not allowed in Strings.pack(). issuedata arrays containing null entries originating from null parameter values in AbstractDeclarativeValidator#error(String string, Integer feature, String code, String... issueData) AbstractDeclarativeValidator#warnString string, Integer feature, String code, String... issueData) Pushed to master. Closing all bugs that were set to RESOLVED before Neon.0 Closing all bugs that were set to RESOLVED before Neon.0 |
an issue array containing null entries creates a NPE inside Strings#pack java.lang.NullPointerException at org.eclipse.xtext.util.Strings.pack(Strings.java:307) at org.eclipse.xtext.ui.editor.validation.MarkerCreator.createMarker(MarkerCreator.java:38) at org.eclipse.xtext.builder.builderState.MarkerUpdaterImpl.addMarkers(MarkerUpdaterImpl.java:108) at org.eclipse.xtext.builder.builderState.MarkerUpdaterImpl.updateMarker(MarkerUpdaterImpl.java:64) at org.eclipse.xtext.builder.builderState.PersistableResourceDescriptionsImpl.doValidate(PersistableResourceDescriptionsImpl.java:154) at org.eclipse.xtext.builder.builderState.PersistableResourceDescriptionsImpl.update(PersistableResourceDescriptionsImpl.java:113) at org.eclipse.xtext.builder.impl.XtextBuilder.doBuild(XtextBuilder.java:127) at org.eclipse.xtext.builder.impl.XtextBuilder.fullBuild(XtextBuilder.java:143) at org.eclipse.xtext.builder.impl.XtextBuilder.build(XtextBuilder.java:73) at org.eclipse.core.internal.events.BuildManager$2.run(BuildManager.java:627) at org.eclipse.core.runtime.SafeRunner.run(SafeRunner.java:42) at org.eclipse.core.internal.events.BuildManager.basicBuild(BuildManager.java:170) at org.eclipse.core.internal.events.BuildManager.basicBuild(BuildManager.java:280) at org.eclipse.core.internal.events.BuildManager.build(BuildManager.java:361) at org.eclipse.core.internal.resources.Project$1.run(Project.java:523) at org.eclipse.core.internal.resources.Workspace.run(Workspace.java:1800) at org.eclipse.core.internal.resources.Workspace.run(Workspace.java:1782) at org.eclipse.core.internal.resources.Project.internalBuild(Project.java:502) at org.eclipse.core.internal.resources.Project.build(Project.java:102) at org.eclipse.xtext.builder.impl.BuildScheduler$BuildJob.run(BuildScheduler.java:97) at org.eclipse.core.internal.jobs.Worker.run(Worker.java:55)