[issue41111] [C API] Convert a few stdlib extensions to the limited C API (PEP 384)

Change by Eric Snow : -- nosy: +eric.snow ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue41111] [C API] Convert a few stdlib extensions to the limited C API (PEP 384)

Change by Christian Heimes : -- nosy: +christian.heimes ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue41111] [C API] Convert a few stdlib extensions to the limited C API (PEP 384)

Change by hai shi : -- versions: +Python 3.11 -Python 3.10 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue41111] [C API] Convert a few stdlib extensions to the limited C API (PEP 384)

hai shi added the comment: > METH_VARARGS and METH_KEYWORDS are part of the stable ABI. Copy that, thanks, petr. -- ___ Python tracker ___

[issue41111] [C API] Convert a few stdlib extensions to the limited C API (PEP 384)

Petr Viktorin added the comment: METH_VARARGS and METH_KEYWORDS are part of the stable ABI. -- ___ Python tracker ___ ___

[issue41111] [C API] Convert a few stdlib extensions to the limited C API (PEP 384)

hai shi added the comment: > The main limitation to use the limited C API for stdlib is Argument Clinic > which attempts to always emit the most efficient code, like using the > METH_FASTCALL calling convention and use private functions like > _PyArg_CheckPositional() or "static

[issue41111] [C API] Convert a few stdlib extensions to the limited C API (PEP 384)

Change by hai shi : -- pull_requests: +24719 pull_request: https://github.com/python/cpython/pull/26080 ___ Python tracker ___ ___

[issue41111] [C API] Convert a few stdlib extensions to the limited C API (PEP 384)

Skip Montanaro added the comment: > > Skip: By the way, I'm curious, why do you use --with-trace-refs? > I'm still horsing around with register opcodes and got in the habit of building with pydebug and trace refs enabled. -- ___ Python tracker

[issue41111] [C API] Convert a few stdlib extensions to the limited C API (PEP 384)

STINNER Victor added the comment: Skip Montanaro: "The latest commit seems to break the build if configured --with-trace-refs." Oops, I forgot about this special build mode. Thanks for the reminder Skip, and thanks for the fix Hai ;-) Skip: By the way, I'm curious, why do you use

[issue41111] [C API] Convert a few stdlib extensions to the limited C API (PEP 384)

STINNER Victor added the comment: New changeset 5787ba4a45492e232f5470c7d2e93763198e4b22 by Hai Shi in branch 'master': bpo-4: Don't build xxlimited with Py_TRACE_REFS macro (GH-25180) https://github.com/python/cpython/commit/5787ba4a45492e232f5470c7d2e93763198e4b22 --

[issue41111] [C API] Convert a few stdlib extensions to the limited C API (PEP 384)

hai shi added the comment: > Perhaps it would be better that compiling them not be attempted with > configuring --with-trace-refs? +1. The limited C API can't be build under Py_TRACE_REFS now. -- ___ Python tracker

[issue41111] [C API] Convert a few stdlib extensions to the limited C API (PEP 384)

Change by hai shi : -- pull_requests: +23922 pull_request: https://github.com/python/cpython/pull/25180 ___ Python tracker ___ ___

[issue41111] [C API] Convert a few stdlib extensions to the limited C API (PEP 384)

Skip Montanaro added the comment: I should revise that comment. The xxlimited and xxlimited_35 modules fail to build. That seems suboptimal, but perhaps is to be expected. Perhaps it would be better that compiling them not be attempted with configuring --with-trace-refs? --

[issue41111] [C API] Convert a few stdlib extensions to the limited C API (PEP 384)

Skip Montanaro added the comment: The latest commit seems to break the build if configured --with-trace-refs. -- nosy: +skip.montanaro ___ Python tracker ___

[issue41111] [C API] Convert a few stdlib extensions to the limited C API (PEP 384)

STINNER Victor added the comment: New changeset 240bcf82a11fe7433a61da70605e924c53b88096 by Victor Stinner in branch 'master': bpo-4: xxlimited.c defines Py_LIMITED_API (GH-25151) https://github.com/python/cpython/commit/240bcf82a11fe7433a61da70605e924c53b88096 --

[issue41111] [C API] Convert a few stdlib extensions to the limited C API (PEP 384)

Change by STINNER Victor : -- pull_requests: +23898 pull_request: https://github.com/python/cpython/pull/25151 ___ Python tracker ___

[issue41111] [C API] Convert a few stdlib extensions to the limited C API (PEP 384)

STINNER Victor added the comment: See also bpo-43688: [C API] Support the limited C API in debug mode (Py_INCREF and Py_DECREF). -- ___ Python tracker ___

[issue41111] [C API] Convert a few stdlib extensions to the limited C API (PEP 384)

hai shi added the comment: > By the way, maybe Py_LIMITED_API should be defined in xxlimited.c, rather > than in setup.py. +1. Defining Py_LIMITED_API in xxlimited.c is more direct than in setup.py. so I created the PR-25115. -- ___ Python

[issue41111] [C API] Convert a few stdlib extensions to the limited C API (PEP 384)

Change by hai shi : -- keywords: +patch pull_requests: +23864 stage: -> patch review pull_request: https://github.com/python/cpython/pull/25115 ___ Python tracker ___

[issue41111] [C API] Convert a few stdlib extensions to the limited C API (PEP 384)

Change by Fred Drake : -- nosy: +fdrake ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe:

[issue41111] [C API] Convert a few stdlib extensions to the limited C API (PEP 384)

Change by STINNER Victor : -- components: +C API title: Convert a few stdlib extensions to the limited C API (PEP 384) -> [C API] Convert a few stdlib extensions to the limited C API (PEP 384) ___ Python tracker