◐ Shell
reader mode source ↗
Skip to content

bpo-35461: Document C API functions which suppress exceptions.#11119

Merged
serhiy-storchaka merged 3 commits into
python:masterfrom
serhiy-storchaka:doc-c-api-suppress-exceptions
Dec 18, 2018
Merged

bpo-35461: Document C API functions which suppress exceptions.#11119
serhiy-storchaka merged 3 commits into
python:masterfrom
serhiy-storchaka:doc-c-api-suppress-exceptions

Conversation

@serhiy-storchaka

@serhiy-storchaka serhiy-storchaka commented Dec 11, 2018 •
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Member

vstinner
vstinner reviewed Dec 11, 2018
View reviewed changes

@vstinner vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hide comment

"Note that errors raised in" is vague. Reader can understand "exceptions directly raised by the executed code", but these functions can get exceptions caused by a signal handler like KeyboardInterrupt.

Maybe add a new paragraph somewhere in the C API documentation to explain that, and add a link to paragraph?

@serhiy-storchaka

serhiy-storchaka commented Dec 17, 2018

Copy link
Copy Markdown
Member Author

The wording was slightly changed.

vstinner
vstinner approved these changes Dec 18, 2018
View reviewed changes

@vstinner vstinner left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Hide comment

LGTM.

@serhiy-storchaka serhiy-storchaka merged commit 3fcc1e0 into python:master Dec 18, 2018
@miss-islington

miss-islington commented Dec 18, 2018

Copy link
Copy Markdown
Contributor

Thanks @serhiy-storchaka for the PR 🌮🎉.. I'm working now to backport this PR to: 2.7, 3.6, 3.7.
🐍🍒⛏🤖

@serhiy-storchaka serhiy-storchaka deleted the doc-c-api-suppress-exceptions branch December 18, 2018 11:57
@bedevere-bot

bedevere-bot commented Dec 18, 2018

Copy link
Copy Markdown

GH-11209 is a backport of this pull request to the 3.7 branch.

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 18, 2018
@serhiy-storchaka @miss-islington
bpo-35461: Document C API functions which suppress exceptions. (pytho… …
a452e19 
…nGH-11119)

(cherry picked from commit 3fcc1e0)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
@bedevere-bot

bedevere-bot commented Dec 18, 2018

Copy link
Copy Markdown

GH-11210 is a backport of this pull request to the 3.6 branch.

@miss-islington

miss-islington commented Dec 18, 2018

Copy link
Copy Markdown
Contributor

Sorry, @serhiy-storchaka, I could not cleanly backport this to 2.7 due to a conflict.
Please backport using cherry_picker on command line.
cherry_picker 3fcc1e08db6fb7e17acc4a8f63be3e42f52f094b 2.7

miss-islington pushed a commit to miss-islington/cpython that referenced this pull request Dec 18, 2018
@serhiy-storchaka @miss-islington
bpo-35461: Document C API functions which suppress exceptions. (pytho… …
8afe996 
…nGH-11119)

(cherry picked from commit 3fcc1e0)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
miss-islington added a commit that referenced this pull request Dec 18, 2018
@miss-islington @serhiy-storchaka
bpo-35461: Document C API functions which suppress exceptions. (GH-11119 …
f265afe 
)

(cherry picked from commit 3fcc1e0)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
@serhiy-storchaka serhiy-storchaka removed their assignment Dec 19, 2018
ned-deily pushed a commit that referenced this pull request Dec 20, 2018
@miss-islington @serhiy-storchaka @ned-deily
bpo-35461: Document C API functions which suppress exceptions. (GH-11119 …
ff740db 
) (GH-11210)

(cherry picked from commit 3fcc1e0)

Co-authored-by: Serhiy Storchaka <storchaka@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@vstinner vstinner vstinner approved these changes

+1 more reviewer

@christopheNan christopheNan christopheNan left review comments

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

docs Documentation in the Doc dir skip news

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

@serhiy-storchaka @miss-islington @bedevere-bot @vstinner @christopheNan @the-knights-who-say-ni