fix some extra cyclic garbage in NurseryManager #3107
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #3107 +/- ##
===================================================
+ Coverage 99.58714% 99.58716% +0.00001%
===================================================
Files 121 121
Lines 18166 18167 +1
Branches 3275 3275
===================================================
+ Hits 18091 18092 +1
Misses 52 52
Partials 23 23
|
|
I'll write a test for this and shape it into a proper PR shortly |
| exc: BaseException | None, | ||
| tb: TracebackType | None, | ||
| ) -> bool: | ||
| del tb |
There was a problem hiding this comment.
This seems intimidating, when you're adding a test could you add another to make sure no stackframes are getting lost? (I assume they're part of exc in addition?)
There was a problem hiding this comment.
The tb is never needed in a context manager, and cpython are looking to deprecate the three arg form in favor of a one arg exc_value only form
| # delete references from locals to avoid creating cycles | ||
| # see test_cancel_scope_exit_doesnt_create_cyclic_garbage | ||
| del _, combined_error_from_nursery, value, new_exc | ||
| del _, combined_error_from_nursery, value, new_exc, exc |
There was a problem hiding this comment.
I think I misunderstood this. There's no cycle here because exc.__traceback__ doesn't see this frame
2 participants
No description provided.