Summary: | Some editor is returning null from getEditorInput() | ||
---|---|---|---|
Product: | [Eclipse Project] JDT | Reporter: | Darin Swanson <Darin_Swanson> |
Component: | Text | Assignee: | JDT-Text-Inbox <jdt-text-inbox> |
Status: | RESOLVED INVALID | QA Contact: | |
Severity: | normal | ||
Priority: | P3 | CC: | n.a.edgar, sxenos |
Version: | 3.1 | Keywords: | needinfo |
Target Milestone: | --- | ||
Hardware: | PC | ||
OS: | Windows XP | ||
Whiteboard: |
Description
Darin Swanson
2005-03-24 01:05:31 EST
It appears that the active editor was returning null as its IEditorInput. This is a bug in the editor, not the problem view... however, the problem view only became sensitive to this type of thing as a result of bug 87288. I've added a paranoia check for null editor inputs, so the problems view is once again protected from buggy editors... However, the real problem is the editor with the bogus input and it's hard to address that without knowing what editor was active at the time. I'm reassigning this to JDT on the general assumption that you had a java editor active at the time... but about the best anyone can do is double-check that there's no obvious way for getEditorInput() to return null and close this as WORKSFORME if they don't find anything. If you notice this again, please indicate what editor had focus at the time the exception was thrown. I've added a check for a non-null editor input at creation time. The workbench will now refuse to create an editor if the editor input is null at creation time. Hopefully this will help catch the culprit. getEditorInput() can return null after the editor has been disposed. But then it shouldn't be firing selection changed events. This could happen if the selection service got a part activation after the part had been disposed. But that shouldn't happen either. Darin, which editor was active? If an editor is open and active the input should not be null. Do you have steps to reproduce? no steps to reproduce. I suspect the null result from getEditorInput() is due to bug 89246. Get rid of deprecated state. . |