Skip to content
This repository has been archived by the owner on Apr 21, 2023. It is now read-only.

Find clean solution for Memory Leak in ShowWhitespaceCharactersActionContributor #12

Closed
cdietrich opened this issue Jul 15, 2016 · 5 comments

Comments

@cdietrich
Copy link
Member

cdietrich commented Jul 15, 2016

Find a clean solution for the workaround in ShowWhitespaceCharactersActionContributor

@cdietrich cdietrich self-assigned this Jul 15, 2016
@cdietrich
Copy link
Member Author

@stadlerb
Copy link

stadlerb commented Jul 30, 2016

The actual leak occurs in Eclipse UI itself, not Xtext UI, so the clean solution must be implemented there. The problem has been known since Eclipse 4.1 and a draft patch has been contributed, but deferred to Eclipse 4.7: https://bugs.eclipse.org/bugs/show_bug.cgi?id=354538

Alternatively, it would be possible to migrate to Eclipse 4 style contributions (for all IActionContributors then, I suppose?), which would be a bit more work.

@cdietrich
Copy link
Member Author

i guess we should stick with the current workaround and wait for the eclipse platform bug to be fixed

@stadlerb
Copy link

stadlerb commented Aug 1, 2016

Ok, but please vote for the Eclipse issue if you have votes left - it's been pretty long-standing and so far, I am the only one who voted.

@cdietrich cdietrich removed their assignment Oct 21, 2016
@cdietrich
Copy link
Member Author

Root Cause is fixed with https://git.eclipse.org/r/#/c/67267/ in Oxygen. Since Xtext will run on older Versions as well i'll close this issue and leave the workaround in place

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants