Skip to content

Conversation

@btharper
Copy link
Contributor

@btharper btharper commented Mar 19, 2019

@btharper
Copy link
Contributor Author

btharper commented Mar 19, 2019

Can someone add a the skip news label for this? This is a further part of bpo-36356 and previously mentioned in Misc/NEWS.d/next/Build/2019-03-18-23-49-15.bpo-36356.WNrwYI.rst

@vstinner
Copy link
Member

Your PR fix maybe the memory leak, but your PR makes me realize that it's a bug. I wrote PR #12454 to fix the bug (and a few other similar bugs). My PR also fix the memory leak.

@vstinner vstinner closed this Mar 20, 2019
@btharper btharper deleted the env_free branch March 20, 2019 01:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants