bpo-43901: Fix refleaks in test_module#25754
Conversation
|
I'm still missing another leak: |
Gotcha! I have it fixed in 3640843 |
|
Ok, we are clean: |
|
I was about to make my PR ;-) |
Sorry, I was doing this patch and missed your comments on BPO :( |
|
It's hard to see how that code could ever have passed a refleak check! It seems like would always have leaked that dict. Yet I swear I ran one! |
|
@larryhastings If you already have the PR made I can close this one and we can merge yours if you prefer :) |
|
I don't mind the parallel effort, as long as it gets fixed one way or another. I've caused enough headaches and gnashing of teeth this week--let's try and keep this incident to a minimum! |
|
Let's go with yours, the PR is already up and checks are running. |
No problem Larry! This happens quite a lot for everyone, and the important thing is that we have your patch for 3.10 :) |
What rot! This never happened during the release process for 3.4 and 3.5! Nobody ever broke the build, ever. I'm clearly a tiresome outlier. |
https://bugs.python.org/issue43901