Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

bpo-36356: Fix leak when updating search path in config#12444

Closed
btharper wants to merge 1 commit into
python:masterpython/cpython:masterfrom
btharper:env_freebtharper/cpython:env_freeCopy head branch name to clipboard
Closed

bpo-36356: Fix leak when updating search path in config#12444
btharper wants to merge 1 commit into
python:masterpython/cpython:masterfrom
btharper:env_freebtharper/cpython:env_freeCopy head branch name to clipboard

Conversation

@btharper

@btharper btharper commented Mar 19, 2019

Copy link
Copy Markdown
Contributor

@btharper

btharper commented Mar 19, 2019

Copy link
Copy Markdown
Contributor Author

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
Copy Markdown
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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.