Skip to content

Fixing memory leak issue#5

Merged
favorart merged 16 commits into
favorart:masterfrom
harris2001:master
Jun 16, 2025
Merged

Fixing memory leak issue#5
favorart merged 16 commits into
favorart:masterfrom
harris2001:master

Conversation

@harris2001

Copy link
Copy Markdown
Contributor

Just a minor patch to fix the memory leak issue when using ScopePtr in VCDVariable and updating Readme.md

  • The original code introduced a circular reference between VarPtr and ScopePtr pointers which was causing memory leak.
  • The updated code uses a non-owning weak_ptr to provide temporary ownership for the ScopePtr pointer and deallocate it after it's been used.

P.S. Thanks again for building this project, it's been very useuful!

@favorart favorart added the bug label Jun 16, 2025
@favorart favorart merged commit 9eb75a4 into favorart:master Jun 16, 2025
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants