[issue38818] Modify PyInterpreterState.eval_frame to pass tstate (PyThreadState)

[STINNER Victor]

Change by STINNER Victor :


--
pull_requests: +18260
pull_request: https://github.com/python/cpython/pull/17340

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue38818] Modify PyInterpreterState.eval_frame to pass tstate (PyThreadState)

[STINNER Victor]

STINNER Victor  added the comment:

I mark this issue as a duplicate of bpo-38500.

--
resolution:  -> duplicate
stage: patch review -> resolved
status: open -> closed
superseder:  -> Provide a way to get/set PyInterpreterState.frame_eval without 
needing to access interpreter internals

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue38818] Modify PyInterpreterState.eval_frame to pass tstate (PyThreadState)

[STINNER Victor]

Change by STINNER Victor :


--
pull_requests: +16837
pull_request: https://github.com/python/cpython/pull/17352

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue38818] Modify PyInterpreterState.eval_frame to pass tstate (PyThreadState)

[Brett Cannon]

Brett Cannon  added the comment:

I think bpo-38500 needs to get resolved first before this as that will affect 
whether this is acceptable or not as an API-breaking change.

--

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue38818] Modify PyInterpreterState.eval_frame to pass tstate (PyThreadState)

[STINNER Victor]

Change by STINNER Victor :


--
keywords: +patch
pull_requests: +16694
stage:  -> patch review
pull_request: https://github.com/python/cpython/pull/17187

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

[issue38818] Modify PyInterpreterState.eval_frame to pass tstate (PyThreadState)

[STINNER Victor]

New submission from STINNER Victor :

Follow-up of bpo-36710 and bpo-38644: I would like to pass explicitly tstate 
(PyThreadState) to internal C functions.

The problem is that PyInterpreterState.eval_frame function has no tstate 
parameter. I propose attached PR to add a tstate parameter. It's a backward 
incompatible change. The "eval_frame" field comes from the PEP 523.

See also bpo-38500 "Provide a way to get/set PyInterpreterState.frame_eval 
without needing to access interpreter internals".

--
components: Interpreter Core
messages: 356736
nosy: brett.cannon, vstinner
priority: normal
severity: normal
status: open
title: Modify PyInterpreterState.eval_frame to pass tstate (PyThreadState)
versions: Python 3.9

___
Python tracker 

___
___
Python-bugs-list mailing list
Unsubscribe: 
https://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com