Skip to content
This repository was archived by the owner on Jan 20, 2022. It is now read-only.

Comments

Actually address the PY_SSIZE_T_CLEAN issue#8

Open
qistoph wants to merge 2 commits intogtrafimenkov:masterfrom
qistoph:master
Open

Actually address the PY_SSIZE_T_CLEAN issue#8
qistoph wants to merge 2 commits intogtrafimenkov:masterfrom
qistoph:master

Conversation

@qistoph
Copy link

@qistoph qistoph commented Jan 20, 2022

Current versions of python report warnings or even errors like this:

DeprecationWarning: PY_SSIZE_T_CLEAN will be required for '#' formats

This PR users the required Py_ssize_t and sets the appropriate PY_SSIZE_T_CLEAN. For details see the first not at https://docs.python.org/3/c-api/arg.html#arg-parsing.

Fixes #5
Improvement on #7, which only sets PY_SSIZE_T_CLEAN, but doesn't actually address the issue.

Also updates the version to 0.1.2.

@gtrafimenkov
Copy link
Owner

Hello @qistoph

Thank you for the PR. Unfortunately I don't have capacity for maintaining this repository. Feel free to fork if you want.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add PY_SSIZE_T_CLEAN to the build as necessary.

2 participants