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 of Client Level #8221

Closed
pietro-lopes opened this issue Aug 28, 2024 · 5 comments
Closed

Memory Leak of Client Level #8221

pietro-lopes opened this issue Aug 28, 2024 · 5 comments
Milestone

Comments

@pietro-lopes
Copy link

Issue description

This is kinda of a soft leak, as it refreshes itself when you join another world.

image

Might want to add a cleanup strategy when single players exit world to clear MekanismClient.

Steps to reproduce

  1. Join world
  2. Leave world
  3. Check memory for leaked objects

Minecraft version

1.21.1 (Latest)

NeoForge version

21.1.23

Mekanism version

10.6.7

Other relevant versions

No response

If a (crash)log is relevant for this issue, link it here: (It's almost always relevant)

No response

@thiakil
Copy link
Member

thiakil commented Sep 5, 2024

this should be fixed with 59645a9

@pietro-lopes
Copy link
Author

Does it cover Client player logout?

@thiakil
Copy link
Member

thiakil commented Sep 6, 2024

It should always have been clearing at that point?

@pietro-lopes
Copy link
Author

pietro-lopes commented Sep 6, 2024

IIRC, Clone does not trigger at logout.
A good rule of thumb is that at main menu, there should be no client level/player, server, etc. instances alive.

@thiakil
Copy link
Member

thiakil commented Sep 6, 2024

Indeed it would not fire, but the existing reset method should have been firing is what I meant

@pupnewfster pupnewfster added this to the 10.7.6 milestone Oct 24, 2024
@thiakil thiakil closed this as completed Dec 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants