FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[3.7] bpo-35615: Fix crashes when copying a Weak{Key,Value}Dictionary. (GH-11384) by miss-islington · Pull Request #11785 · python/cpython · GitHub

Repository navigation

[3.7] bpo-35615: Fix crashes when copying a Weak{Key,Value}Dictionary. (GH-11384) - #11785

Merged
pitrou merged 1 commit into
python:3.7from
miss-islington:backport-96d37db-3.7
Feb 7, 2019
Merged

pitrou merged 1 commit into
python:3.7from
miss-islington:backport-96d37db-3.7

Conversation

miss-islington commented Feb 7, 2019 •
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Contributor

Protect dict iterations by wrapping them with _IterationGuard in the
following methods:

  • WeakValueDictionary.copy()
  • WeakValueDictionary.deepcopy()
  • WeakKeyDictionary.copy()
  • WeakKeyDictionary.deepcopy()
    (cherry picked from commit 96d37db)

Co-authored-by: Fish ltfish@users.noreply.github.com

https://bugs.python.org/issue35615

…honGH-11384)

Protect dict iterations by wrapping them with _IterationGuard in the
following methods:

- WeakValueDictionary.copy()
- WeakValueDictionary.__deepcopy__()
- WeakKeyDictionary.copy()
- WeakKeyDictionary.__deepcopy__()
(cherry picked from commit 96d37db)

Co-authored-by: Fish <ltfish@users.noreply.github.com>

Copy link
Copy Markdown
Contributor Author

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

pitrou merged commit 48769a2 into python:3.7 Feb 7, 2019
miss-islington deleted the backport-96d37db-3.7 branch February 7, 2019 20:09

Copy link
Copy Markdown
Contributor Author

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

This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type-bug An unexpected behavior, bug, or error

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants


Back | FazBrowse Home | New Git URL