Skip to content

bpo-41333: Convert OrderedDict.pop() to Argument Clinic - #21534

Merged
serhiy-storchaka merged 3 commits into
python:masterfrom
serhiy-storchaka:clinic-odict-pop
Jul 19, 2020
Merged

bpo-41333: Convert OrderedDict.pop() to Argument Clinic#21534
serhiy-storchaka merged 3 commits into
python:masterfrom
serhiy-storchaka:clinic-odict-pop

Conversation

@serhiy-storchaka

@serhiy-storchaka serhiy-storchaka commented Jul 18, 2020

Copy link
Copy Markdown
Member

Comment thread Objects/odictobject.c Outdated

Remove specified key and return the corresponding value.

If key is not found, default is returned if given, otherwise KeyError

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

We can make this read a little better, "If the key is not found, return the default if given; otherwise, raise a KeyError."

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Thank you for your suggestion.

@serhiy-storchaka
serhiy-storchaka merged commit 6bf3237 into python:master Jul 19, 2020
@serhiy-storchaka
serhiy-storchaka deleted the clinic-odict-pop branch July 19, 2020 06:19
shihai1991 pushed a commit to shihai1991/cpython that referenced this pull request Aug 4, 2020
shihai1991 pushed a commit to shihai1991/cpython that referenced this pull request Aug 20, 2020
xzy3 pushed a commit to xzy3/cpython that referenced this pull request Oct 18, 2020
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.

5 participants