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

Memory leak #2

Open
nachoburr opened this issue Apr 17, 2023 · 4 comments
Open

Memory leak #2

nachoburr opened this issue Apr 17, 2023 · 4 comments

Comments

@nachoburr
Copy link

Hi, this is working well but creates zombies.

@WilhelmOks
Copy link
Owner

I'll take a look at it.

@WilhelmOks
Copy link
Owner

I checked it with the profiler and it looks like the VEdge is the problem. But I can't find an obvious reason why.

@nachoburr
Copy link
Author

nachoburr commented Apr 22, 2023 via email

@WilhelmOks
Copy link
Owner

I used the Zombie profiler but from what I have read, Zombies are over-released objects, so they wouldn't cause memory leaks.

What I found is, that many objects stay in memory and are never released even if the screen is reset it the test app.
The first object that was directly from the lib was VEdge, so this is a possible candidate for memory leaks but it could also be something else and VEdge is just a symptom instead of the cause.

There is also the chance that the leak is actually in the sample app and not in the library itself.

I'm not very experienced in analyzing that kind of stuff and the fact that I more or less blindly copied the logic from the original C# lib, which has garbage collection, also makes it difficult. It was also more than 3 years ago.

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

2 participants