-
Notifications
You must be signed in to change notification settings - Fork 79
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
NPE in PDEModelUtility.java:415 #1404
Comments
May be we could consider this issue as a Good First issue |
I'm working on this |
Great. You are one of the students from CodeDays, aren't you? Much success on this task and don't hesitate to ask for help if you need any. |
Hi @jukzi , I tried to reproduce the issue in both the most recent public release (2024.09) of Eclipse and in the development version. I opened 90 Eclipse project files and used “Close Others”. I didn’t get any kind of error. I also let it sit for a bit, and still didn’t get an error. Based on this, I’m wondering if this is still a problem for you or if it was fixed somehow? If this is still an issue for you, could you give us more details such as log files, specifics on the type of editors you opened, etc. I’d be especially interested in a complete minimal reproduction. Thank you 👍 |
I don't have more information then already provided. Such race conditions may be hard to reproduce. |
Hi all, I'm one of the Consulting Software Engineers on the general mentor team at CodeDay Labs/Init and have worked with @slvo-loading extensively on this issue. Given that @jukzi (thank you for the quick response on such an old issue, by the way!) cannot easily identify a minimal set of reproduction steps for this bug and believes it might be a "race condition," I would like to suggest that the |
hint: this type of errors can often be solved by thinking about what could have gone wrong. An intermediate step may involve adding a check at an earlier point of time or extending errormessage/logging to get missing information. |
As far as i remember the last action i did before the error was logged: Close like 90 editors by "Close Others" from the editor tab context menu.
As far as i see in the code org.eclipse.pde.internal.ui.editor.PDEFormEditor.dispose() is the only thing that makes contextManager = null.
The text was updated successfully, but these errors were encountered: