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

Jump to bottom

bpo-45474: Exclude all of marshal.h if Py_LIMITED_API is defined #29061

Merged
merged 2 commits into from Oct 20, 2021
Merged

bpo-45474: Exclude all of marshal.h if Py_LIMITED_API is defined #29061

merged 2 commits into from Oct 20, 2021

Commits on Oct 19, 2021

  1. Reword note on removing PyMarshal_* with Py_LIMITED_API 

    These items were not part of the limited API, which is defined
in the docs (via Misc/stable_abi.txt).

The change brings the `Py_LIMITED_API` macro closer to the ideal
of only allowing things in the limited API.
    @encukou
    encukou committed Oct 19, 2021
    9a7098f

  2. Exclude all of "marshal.h" when Py_LIMITED_API is defined. 

    Nothing in the file is listed as part of the limited API.
The symbols are not exported in the Windows stable ABI dlls.
The header is not included from <Python.h>.
    @encukou
    encukou committed Oct 19, 2021
    7fc9eef