Skip to content

Fix leaked resources identified by StrictMode#59

Merged
marandaneto merged 3 commits into
mainfrom
chore/fix-leaks
Nov 7, 2023
Merged

Fix leaked resources identified by StrictMode#59
marandaneto merged 3 commits into
mainfrom
chore/fix-leaks

Conversation

@marandaneto
Copy link
Copy Markdown
Member

💡 Motivation and Context

StrictMode revealed that some resources were not closed.
Also added a memory leak detector

💚 How did you test it?

📝 Checklist

  • I reviewed the submitted code.
  • I added tests to verify the changes.
  • I updated the docs if needed.
  • No breaking change or entry added to the changelog.

Copy link
Copy Markdown
Member

@pauldambra pauldambra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🙌

@marandaneto marandaneto merged commit 54e9267 into main Nov 7, 2023
@marandaneto marandaneto deleted the chore/fix-leaks branch November 7, 2023 16:09
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.

2 participants