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

bpo-38500: Add PyInterpreterState_SetEvalFrameFunc() by vstinner · Pull Request #17352 · python/cpython · GitHub

Repository navigation

bpo-38500: Add PyInterpreterState_SetEvalFrameFunc() - #17352

Closed
vstinner wants to merge 3 commits into
python:masterfrom
vstinner:set_eval_frame_tstate
Closed

vstinner wants to merge 3 commits into
python:masterfrom
vstinner:set_eval_frame_tstate

Conversation

vstinner commented Nov 22, 2019 •
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Member

Add a public API to get and set the frame evaluation function:

  • Rename the internal _PyFrameEvalFunction type to
    PyFrameEvalFunction and make it public
  • Add PyInterpreterState_GetEvalFrameFunc() and
    PyInterpreterState_SetEvalFrameFunc() functions

bpo-38818: PyInterpreterState.eval_frame now pass tstate

PyInterpreterState.eval_frame (PEP 523) now requires a new mandatory
tstate parameter (PyThreadState*).

https://bugs.python.org/issue38500

Copy link
Copy Markdown
Member Author

This is PR #17340 merged with PR #17187: add PyInterpreterState_SetEvalFrameFunc() and add tstate parameter to eval_frame.

Comment thread Doc/c-api/init.rst Outdated
Add a public API to get and set the frame evaluation function:

* Rename the internal _PyFrameEvalFunction type to
  PyFrameEvalFunction and make it public
* Add PyInterpreterState_GetEvalFrameFunc() and
  PyInterpreterState_SetEvalFrameFunc() functions
PyInterpreterState.eval_frame (PEP 523) now requires a new mandatory
tstate parameter (PyThreadState*).

Copy link
Copy Markdown
Member Author

Rebase to fix a conflict.

Copy link
Copy Markdown
Member Author

I abandon this PR in favor of PR #17340.

vstinner closed this Mar 10, 2020
vstinner deleted the set_eval_frame_tstate branch March 10, 2020 18:05
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL