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.

Sign up for GitHub

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

Jump to bottom

[Android] Memory leak on JmeSurfaceView #2010

Open
pavly-gerges opened this issue May 15, 2023 · 0 comments · May be fixed by #2017
Open

[Android] Memory leak on JmeSurfaceView #2010

pavly-gerges opened this issue May 15, 2023 · 0 comments · May be fixed by #2017
Labels
Android bug Something that is supposed to work, but doesn't. More severe than a "defect".
Milestone
Future Release

Comments

@pavly-gerges
Copy link
Contributor

According to this thread, there is a memory leak, the test was conducted on JmeSurfaceView.

A proposed blind solution was to add JmeSystemDelegate.setView(null) within the JmeSurfaceView#destroy(), which surprisingly was already on AndroidHarness, I think this is a real memory leak that is specific to JmeSurfaceView.
@pavly-gerges pavly-gerges changed the title [Android] Memory leak [Android] Memory leak on JmeSurfaceView May 15, 2023
@Ali-RS Ali-RS added bug Something that is supposed to work, but doesn't. More severe than a "defect". Android labels May 15, 2023
@Ali-RS Ali-RS added this to the Future Release milestone May 15, 2023
@pavly-gerges pavly-gerges linked a pull request May 21, 2023 that will close this issue
Draft
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Android bug Something that is supposed to work, but doesn't. More severe than a "defect".
Projects
None yet
Milestone
Future Release
Development

Successfully merging a pull request may close this issue.

JmeSurfaceView: fix memory leak Electrostat-Lab/jmonkeyengine
2 participants
@Ali-RS @pavly-gerges