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

[Eric Snow]

Change by Eric Snow :


--
nosy: +eric.snow

___
Python tracker 

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

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

[Christian Heimes]

Change by Christian Heimes :


--
nosy: +christian.heimes

___
Python tracker 

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

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

[hai shi]

Change by hai shi :


--
versions: +Python 3.11 -Python 3.10

___
Python tracker 

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

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

[hai shi]

hai shi  added the comment:

> METH_VARARGS and METH_KEYWORDS are part of the stable ABI.
  Copy that, thanks, petr.

--

___
Python tracker 

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

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

[Petr Viktorin]

Petr Viktorin  added the comment:

METH_VARARGS and METH_KEYWORDS are part of the stable ABI.

--

___
Python tracker 

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

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

[hai shi]

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 _PyArg_Parser _parser".

PR-26080 adds the feature that Argument Clinic to support to use the Limited C 
API.
METH_FASTCALL is the part of the stable ABI in PR-23009.
Do we need convert METH_VARARGS and METH_KEYWORDS as the part of the stable ABI 
too(something like METH_FASTCALL)?

--

___
Python tracker 

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

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

[hai shi]

Change by hai shi :


--
pull_requests: +24719
pull_request: https://github.com/python/cpython/pull/26080

___
Python tracker 

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

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

[Skip Montanaro]

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 

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

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

[STINNER Victor]

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 --with-trace-refs?

--

___
Python tracker 

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

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

[STINNER Victor]

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


--

___
Python tracker 

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

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

[hai shi]

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 

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

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

[hai shi]

Change by hai shi :


--
pull_requests: +23922
pull_request: https://github.com/python/cpython/pull/25180

___
Python tracker 

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

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

[Skip Montanaro]

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?

--

___
Python tracker 

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

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

[Skip Montanaro]

Skip Montanaro  added the comment:

The latest commit seems to break the build if configured --with-trace-refs.

--
nosy: +skip.montanaro

___
Python tracker 

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

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

[STINNER Victor]

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


--

___
Python tracker 

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

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

[STINNER Victor]

Change by STINNER Victor :


--
pull_requests: +23898
pull_request: https://github.com/python/cpython/pull/25151

___
Python tracker 

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

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

[STINNER Victor]

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 

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

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

[hai shi]

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 tracker 

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

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

[hai shi]

Change by hai shi :


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

___
Python tracker 

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

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

[Fred Drake]

Change by Fred Drake :


--
nosy: +fdrake

___
Python tracker 

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

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

[STINNER Victor]

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 

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