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

bpo-29684: Fix regression of PyEval_CallObjectWithKeywords #87

Merged
merged 2 commits into from
Mar 1, 2017

Conversation

methane
Copy link
Member

@methane methane commented Feb 14, 2017

It should raise TypeError when kwargs is not a dict.

@methane methane requested a review from vstinner February 14, 2017 04:29
@methane methane changed the base branch from 3.6 to master March 1, 2017 09:23
@methane methane changed the title bpo-29548: Fix regression of PyEval_CallObjectWithKeywords bpo-29684: Fix regression of PyEval_CallObjectWithKeywords Mar 1, 2017
It should raise TypeError when kwargs is not a dict.
@methane methane merged commit 3824cd8 into python:master Mar 1, 2017
@methane methane deleted the fix-evalcall branch March 1, 2017 11:41
akruis pushed a commit to akruis/cpython that referenced this pull request Sep 9, 2017
Never call inter-thread scheduler for a tasklet switch with prev==next.
This fixes the first part of issue python#87.
Fix an assert, that is no longer valid. Fixes part 2 of issue python#87.
Add a test case for two assertion failures during interpreter shutdown.

https://bitbucket.org/stackless-dev/stackless/issues/87
(grafted from a6e5950cb45fc1d2d101866c20aea7eb5f005db0, 73f4ac31aa6a and
9f4606055d9c)
colesbury referenced this pull request in colesbury/nogil Oct 6, 2021
There was a period between detaching the PyThreadState on
thread exit and abandoning the heap where a GC thread might
see an inconsistent state of the heap.

See #87
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
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