Skip to content

Add assert(PyGILState_Check()) in gil_scoped_release constructor. - #30067

Closed
rwgk wants to merge 1 commit into
google:mainfrom
rwgk:scoped_gil_release_gil_check_pywrapcc
Closed

Add assert(PyGILState_Check()) in gil_scoped_release constructor.#30067
rwgk wants to merge 1 commit into
google:mainfrom
rwgk:scoped_gil_release_gil_check_pywrapcc

Conversation

@rwgk

@rwgkrwgk commented Oct 5, 2023

Copy link
Copy Markdown
Contributor

Description

Safety guard, to prevent undefined behavior (in the general sense).

Suggested changelog entry:

@rwgk

rwgk commented Oct 12, 2023

Copy link
Copy Markdown
ContributorAuthor

This one-line change is included in pybind/pybind11#4877.

@rwgkrwgk closed this Oct 12, 2023
@rwgk
rwgk deleted the scoped_gil_release_gil_check_pywrapcc branch October 12, 2023 04:37
Sign up for freeto 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.

1 participant

@rwgk