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

bpo-32500: Fix error messages for sequence and mapping C API. by serhiy-storchaka · Pull Request #7846 · python/cpython · GitHub

/ cpython Public

bpo-32500: Fix error messages for sequence and mapping C API. - #7846

Merged
serhiy-storchaka merged 1 commit into
python:masterfrom
serhiy-storchaka:mapping-sequence-api-errors
Jul 23, 2018
Merged

bpo-32500: Fix error messages for sequence and mapping C API.#7846
serhiy-storchaka merged 1 commit into
python:masterfrom
serhiy-storchaka:mapping-sequence-api-errors

Conversation

serhiy-storchaka commented Jun 21, 2018
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Member

Fix error messages for PySequence_Size(), PySequence_GetItem(),
PySequence_SetItem() and PySequence_DelItem() called with a mapping
and PyMapping_Size() called with a sequence.

https://bugs.python.org/issue32500

Fix error messages for PySequence_Size(), PySequence_GetItem(),
PySequence_SetItem() and PySequence_DelItem() called with a mapping
and PyMapping_Size() called with a sequence.

ammaraskar commented Jun 23, 2018
edited
Loading

Copy link
Copy Markdown
Member

Shouldn't this be attached to https://bugs.python.org/issue33933 which is specifically about the error message?

Edit: nevermind, Raymond addresses this on the ticket.

Copy link
Copy Markdown
Member Author

I'm going to merge this PR if there are no other suggestions.

serhiy-storchaka merged commit a6fdddb into python:master Jul 23, 2018
serhiy-storchaka deleted the mapping-sequence-api-errors branch July 23, 2018 20:43
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

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL