Skip to content

[3.8] bpo-38006: Avoid closure in weakref.WeakValueDictionary (GH-15641)#15787

Merged
miss-islington merged 1 commit intopython:3.8from
miss-islington:backport-a2af05a-3.8
Sep 9, 2019
Merged

[3.8] bpo-38006: Avoid closure in weakref.WeakValueDictionary (GH-15641)#15787
miss-islington merged 1 commit intopython:3.8from
miss-islington:backport-a2af05a-3.8

Conversation

@miss-islington
Copy link
Copy Markdown
Contributor

@miss-islington miss-islington commented Sep 9, 2019

weakref.WeakValueDictionary defines a local remove() function used as
callback for weak references. This function was created with a
closure. Modify the implementation to avoid the closure.
(cherry picked from commit a2af05a)

Co-authored-by: Victor Stinner vstinner@redhat.com

https://bugs.python.org/issue38006

weakref.WeakValueDictionary defines a local remove() function used as
callback for weak references. This function was created with a
closure.  Modify the implementation to avoid the closure.
(cherry picked from commit a2af05a)

Co-authored-by: Victor Stinner <vstinner@redhat.com>
Copy link
Copy Markdown
Member

@vstinner vstinner left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, good bot.

@miss-islington
Copy link
Copy Markdown
Contributor Author

@vstinner: Status check is done, and it's a success ✅ .

@miss-islington miss-islington merged commit 78d15fa into python:3.8 Sep 9, 2019
@miss-islington miss-islington deleted the backport-a2af05a-3.8 branch September 9, 2019 16:24
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.

4 participants