Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bpo28598 backport 3.6 #92

Closed
wants to merge 2 commits into from
Closed

Bpo28598 backport 3.6 #92

wants to merge 2 commits into from

Conversation

mjpieters
Copy link
Contributor

Backport for #51

@mjpieters mjpieters closed this Feb 14, 2017
@mjpieters mjpieters deleted the bpo28598_backport_3.6 branch February 14, 2017 09:07
akruis pushed a commit to akruis/cpython that referenced this pull request Sep 9, 2017
tasklet.bind() an tasklet.bind_thread() now correctly use the nesting level
to test for C-state. Additionally I renamed the static method
tasklet_has_c_stack() to tasklet_has_c_stack_and_thread().
The test in  tasklet_has_c_stack() was not correct, if the
tasklet was the current tasklet of its thread. (GC indirectly
calls this method for all tasklets from arbitrary threads.)

Additional related changes:
- This commit changes an assertion to allow deallocation of tasklets with
a non zero C-stack but no tstate.
- The commit adds a warning message about deallocation tasklets with non
zero C-stack, if Py_VerboseFlag is non zero.
- On deallocation of a tasklet, zero the "task" pointer of its cstate.

The commit adds a test case for tasklet.bind() and tasklet.bind_thread()
on a tasklet, that has C-state but no thread-state.

https://bitbucket.org/stackless-dev/stackless/issues/92
(grafted from 7c71dffe45072a070aa59d9c97b593d89d0be18e, b79ce9e731a5,
2a10937f0ab9, a5c487b3bd51, b171a8b3ee7d and b45e6e4786f9)
akruis pushed a commit to akruis/cpython that referenced this pull request Sep 9, 2017
jaraco pushed a commit that referenced this pull request Dec 2, 2022
jaraco added a commit to jaraco/cpython that referenced this pull request Feb 17, 2023
jaraco added a commit to jaraco/cpython that referenced this pull request Feb 17, 2023
Lower standards to only 70% coverage and run tests on Python 3.7 and 3.8.

Closes python#92

See merge request python-devs/importlib_resources!100
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.

3 participants