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: properly unset variables after use in SocketServer #21

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jenksed
Copy link

@jenksed jenksed commented Jan 16, 2025

fix: properly unset variables after use in SocketServer

  • Add strategic variable cleanup in loopOnce() to prevent memory leaks
  • Clean up loop variables in triggerHooks() after each iteration
  • Improve memory management in shutDownEverything() with explicit cleanup
  • Add variable cleanup in removeHook()
  • Handle variable cleanup before early returns

This commit addresses memory leaks by ensuring all variables are properly
unset when they are no longer needed, particularly in long-running server
operations and loops.

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

Successfully merging this pull request may close these issues.

1 participant