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

bpo-35081: Move _PyGC_FINALIZED() back to C API by vstinner · Pull Request #10626 · python/cpython · GitHub

/ cpython Public

bpo-35081: Move _PyGC_FINALIZED() back to C API - #10626

Merged
vstinner merged 1 commit into
python:masterfrom
vstinner:_PyGC_FINALIZED
Nov 20, 2018
Merged

bpo-35081: Move _PyGC_FINALIZED() back to C API#10626
vstinner merged 1 commit into
python:masterfrom
vstinner:_PyGC_FINALIZED

Conversation

vstinner commented Nov 20, 2018
edited by bedevere-bot
Loading

Copy link
Copy Markdown
Member

Partially revert commit 1a6be91,
move back PyGC API from the internal API to the C API:

  • _PyGCHead_NEXT(g), _PyGCHead_SET_NEXT(g, p)
  • _PyGCHead_PREV(g), _PyGCHead_SET_PREV(g, p)
  • _PyGCHead_FINALIZED(g), _PyGCHead_SET_FINALIZED(g)
  • _PyGC_FINALIZED(o), _PyGC_SET_FINALIZED(o)
  • _PyGC_PREV_MASK_FINALIZED
  • _PyGC_PREV_MASK_COLLECTING
  • _PyGC_PREV_SHIFT
  • _PyGC_PREV_MASK

_PyObject_GC_TRACK(o) and _PyObject_GC_UNTRACK(o) remain in the
internal API.

https://bugs.python.org/issue35081

Partially revert commit 1a6be91,
move back PyGC API from the internal API to the C API:

* _PyGCHead_NEXT(g), _PyGCHead_SET_NEXT(g, p)
* _PyGCHead_PREV(g), _PyGCHead_SET_PREV(g, p)
* _PyGCHead_FINALIZED(g), _PyGCHead_SET_FINALIZED(g)
* _PyGC_FINALIZED(o), _PyGC_SET_FINALIZED(o)
* _PyGC_PREV_MASK_FINALIZED
* _PyGC_PREV_MASK_COLLECTING
* _PyGC_PREV_SHIFT
* _PyGC_PREV_MASK

_PyObject_GC_TRACK(o) and _PyObject_GC_UNTRACK(o) remain in the
internal API.

AvdN commented Dec 12, 2018
edited
Loading

Copy link
Copy Markdown

This fixed an issue with the C based loader for YAML in ruamel.yaml as well (which is Cython based).

Copy link
Copy Markdown
Member Author

This fixed an issue with the C based loaded for YAML in ruamel.yaml as well (which is Cython based).

Oh, good to know. Thanks ;-)

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