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

[3.6] bpo-20891: Fix PyGILState_Ensure() (#4650) by vstinner · Pull Request #4655 · python/cpython · GitHub

/ cpython Public

[3.6] bpo-20891: Fix PyGILState_Ensure() (#4650) - #4655

Merged
vstinner merged 1 commit into
python:3.6from
vstinner:pygilstate_ensure36
Nov 30, 2017
Merged

[3.6] bpo-20891: Fix PyGILState_Ensure() (#4650)#4655
vstinner merged 1 commit into
python:3.6from
vstinner:pygilstate_ensure36

Conversation

vstinner commented Nov 30, 2017
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Member

When PyGILState_Ensure() is called in a non-Python thread before
PyEval_InitThreads(), only call PyEval_InitThreads() after calling
PyThreadState_New() to fix a crash.

Add an unit test in test_embed.

Enhance also embedded tests, backport from master:

  • Add test_pre_initialization_api()
  • Set PYTHONIOENCODING environment variable in
    test_forced_io_encoding()

(cherry picked from commit b4d1e1f)

https://bugs.python.org/issue20891

When PyGILState_Ensure() is called in a non-Python thread before
PyEval_InitThreads(), only call PyEval_InitThreads() after calling
PyThreadState_New() to fix a crash.

Add an unit test in test_embed.

Enhance also embedded tests, backport from master:

* Add test_pre_initialization_api()
* Set PYTHONIOENCODING environment variable in
  test_forced_io_encoding()

(cherry picked from commit b4d1e1f)
vstinner changed the title [3.6] bpo-20891: Fix PyGILState_Ensure() (#4650) [3.6] Fix PyGILState_Ensure() (#4650) Nov 30, 2017
vstinner changed the title [3.6] Fix PyGILState_Ensure() (#4650) [3.6] bpo-20891: Fix PyGILState_Ensure() (#4650) Nov 30, 2017
vstinner merged commit e10c9de into python:3.6 Nov 30, 2017
vstinner deleted the pygilstate_ensure36 branch November 30, 2017 22:36
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.

1 participant


Back | FazBrowse Home | New Git URL