Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

gh-104922: MakePY_SSIZE_T_CLEAN not mandatory again#105051

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
methane merged 10 commits intopython:mainfrommethane:always-ssize_t_clean
May 31, 2023

Conversation

methane
Copy link
Member

@methanemethane commentedMay 29, 2023
edited
Loading

This PR makes APIs like PyArg_Parse() accepts# format specifier with ssize_t.
_SizeT variants APIs are now ABI-only if they are part of stable ABI.

  • Update docs
  • Add news entry.
  • Add what's new entry.

@methane
Copy link
MemberAuthor

@vstinner I prefer this PR (PyArg_ParseTuple() now uses Py_ssize_t) over#104923 (PyArg_ParseTuple() keeps raising SystemError for '#' but always#define PyArg_ParseTuple _PyArg_ParseTuple_SizeT.

What do you think?

@methanemethaneforce-pushed thealways-ssize_t_clean branch from954122c toa0af3b5CompareMay 30, 2023 12:54
@vstinner
Copy link
Member

What do you think?

I'm away from Python. You will have to find a different reviewer 😁

@methane
Copy link
MemberAuthor

I removed NEWS entry because current CI workflow doesn't accept path including space.

#103914 (comment)

@methanemethaneforce-pushed thealways-ssize_t_clean branch from238cd95 tocb35202CompareMay 31, 2023 01:07
@methanemethane marked this pull request as ready for reviewMay 31, 2023 04:06
@methanemethane merged commitadccff3 intopython:mainMay 31, 2023
@methanemethane deleted the always-ssize_t_clean branchMay 31, 2023 09:38
yselkowitz added a commit to yselkowitz/vim that referenced this pull requestJun 23, 2024
_PyObject_CallFunction_SizeT was dropped in Python 3.13:python/cpython#105051Signed-off-by: Yaakov Selkowitz <yselkowi@redhat.com>
chrisbra pushed a commit to vim/vim that referenced this pull requestJul 4, 2024
…hon 3.13Problem:  if_py: _PyObject_CallFunction_SizeT is dropped in Python 3.13Solution: define PyObject_CallFunction for Python >= 3.13          (Yaakov Selkowitz)references:python/cpython#105051closes:#15088Signed-off-by: Yaakov Selkowitz <yselkowi@redhat.com>Signed-off-by: Christian Brabandt <cb@256bit.org>
RestorerZ pushed a commit to RestorerZ/fork_vim-orig that referenced this pull requestJul 9, 2024
…hon 3.13Problem:  if_py: _PyObject_CallFunction_SizeT is dropped in Python 3.13Solution: define PyObject_CallFunction for Python >= 3.13          (Yaakov Selkowitz)references:python/cpython#105051closes:vim#15088Signed-off-by: Yaakov Selkowitz <yselkowi@redhat.com>Signed-off-by: Christian Brabandt <cb@256bit.org>
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment
Reviewers

@markshannonmarkshannonAwaiting requested review from markshannonmarkshannon is a code owner

Assignees
No one assigned
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

3 participants
@methane@vstinner@bedevere-bot

[8]ページ先頭

©2009-2025 Movatter.jp