Skip to content
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

Fix memory leak in settings component #8

Open
bluelhf opened this issue Feb 25, 2023 · 0 comments
Open

Fix memory leak in settings component #8

bluelhf opened this issue Feb 25, 2023 · 0 comments

Comments

@bluelhf
Copy link
Collaborator

bluelhf commented Feb 25, 2023

According to the plugin verifier, we leak memory in a lambda in SettingsComponent, as a registered Disposable object is never disposed of:

SEVERE - #c.i.o.u.ObjectTree - Memory leak detected: 'fi.testaustime.plugin_intellij.configuration.SettingsComponent$$Lambda$1549/0x00000001012587b8@1c4fc60' of class fi.testaustime.plugin_intellij.configuration.SettingsComponent$$Lambda$1549/0x00000001012587b8 is registered in Disposer but wasn't disposed.
Register it with a proper parentDisposable or ensure that it's always disposed by direct Disposer.dispose call.
See https://jetbrains.org/intellij/sdk/docs/basics/disposers.html for more details.

We should dispose of the resource properly to avoid leaking memory.

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

No branches or pull requests

1 participant