net-lisias-ksp/DistantObject

Memory leak from settings window

Closed this issue · 1 comments

I see this a lot in people's logs:

[KSPCF:MemoryLeaks] A destroyed DistantObject:SettingsGui instance is owning a onGUIApplicationLauncherDestroyed GameEvents callback. No action has been taken, but unless this mod is relying on this pattern, this is likely a memory leak.

The settings gui class never unhooks its event handler so it never gets garbage collected.

Lisias commented

@JonnyOThan , sorry for being so late. I just detected this while doing Support on an unrelated problem and when I came back here to publish the release, I got aware of your issue.

Anyway, thank you for the report and my apologies for not detecting it sooner - may I ask you to next time mention me (@Lisias) on the issue? This way github will warn me about and I will not miss it, as it happened today!

In a way or another, it's fixed and available on the Releases page!