Repository navigation
Use only public C-API in _xxsubinterpreters module. #80278
Description
Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.
Show more details
GitHub fields:
assignee = 'https://github.com/ericsnowcurrently'
closed_at = <Date 2019-03-29.21:09:52.140>
created_at = <Date 2019-02-23.23:48:58.098>
labels = ['interpreter-core', '3.8']
title = 'Use only public C-API in _xxsubinterpreters module.'
updated_at = <Date 2019-03-29.21:09:52.139>
user = 'https://github.com/ericsnowcurrently'bugs.python.org fields:
activity = <Date 2019-03-29.21:09:52.139>
actor = 'eric.snow'
assignee = 'eric.snow'
closed = True
closed_date = <Date 2019-03-29.21:09:52.140>
closer = 'eric.snow'
components = ['Interpreter Core']
creation = <Date 2019-02-23.23:48:58.098>
creator = 'eric.snow'
dependencies = []
files = []
hgrepos = []
issue_num = 36097
keywords = ['patch']
message_count = 6.0
messages = ['336419', '336973', '337114', '337214', '337216', '338042']
nosy_count = 2.0
nosy_names = ['vstinner', 'eric.snow']
pr_nums = ['12003', '12359']
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue36097'
versions = ['Python 3.8']
Activity
ericsnowcurrently commentedon Feb 23, 2019
After discussions about our use of the public C-API in stdlib extension modules, I realized that I'd written the _xxsubinterpreters module using internal C-API. However, there's no reason not to stick to the public C-API. Fixing this will require adding a few new "private" functions.
ericsnowcurrently commentedon Mar 1, 2019
New changeset bcfa450 by Eric Snow in branch 'master':
bpo-36097: Use only public C-API in the_xxsubinterpreters module (adding as necessary). (bpo-12003)
bcfa450
vstinner commentedon Mar 4, 2019
New changeset 4d61e6e by Victor Stinner in branch 'master':
Revert: bpo-33608: Factor out a private, per-interpreter _Py_AddPendingCall(). (GH-11617) (GH-12159)
4d61e6e
ericsnowcurrently commentedon Mar 5, 2019
I'll re-merge this once this problem in issue bpo-33608 is resolved.
ericsnowcurrently commentedon Mar 5, 2019
Actually, this is independent of that change. It had to be reverted because the PR was based on the earlier PR from bpo-33608. So I may merge this separately...or not, since it would mean having to sort out merge conflicts. :)
ericsnowcurrently commentedon Mar 15, 2019
New changeset c11183c by Eric Snow in branch 'master':
bpo-36097: Use only public C-API in the_xxsubinterpreters module (adding as necessary). (gh-12359)
c11183c