Skip to content

Fix issue around closure and beartype causing memory leaks - #324

Merged
benoit74 merged 1 commit into
mainfrom
fix_mem_leaks_2
Jul 30, 2026
Merged

Fix issue around closure and beartype causing memory leaks#324
benoit74 merged 1 commit into
mainfrom
fix_mem_leaks_2

Conversation

@benoit74

@benoit74 benoit74 commented Jul 30, 2026

Copy link
Copy Markdown
Collaborator

One more fix for #322

Fix reference cycle in RxRewriter.rewrite caused by beartype_this_package() re-decorating a per-call closure, permanently pinning rewriter objects in memory

I've opened an issue in beartype repo as well: beartype/beartype#673

@benoit74 benoit74 self-assigned this Jul 30, 2026
@codecov

codecov Bot commented Jul 30, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.96%. Comparing base (c23b959) to head (e784d56).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #324   +/-   ##
=======================================
  Coverage   99.96%   99.96%           
=======================================
  Files          41       41           
  Lines        2558     2559    +1     
  Branches      364      364           
=======================================
+ Hits         2557     2558    +1     
  Misses          1        1           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@benoit74
benoit74 merged commit fd769ef into main Jul 30, 2026
10 checks passed
@benoit74
benoit74 deleted the fix_mem_leaks_2 branch July 30, 2026 19:47
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