X Tutup
The Wayback Machine - https://web.archive.org/web/20260219034057/https://github.com/python/cpython/pull/118010
Skip to content

gh-117873: Deprecate grp keyword arguments#118010

Closed
vstinner wants to merge 1 commit intopython:mainfrom
vstinner:grp
Closed

gh-117873: Deprecate grp keyword arguments#118010
vstinner wants to merge 1 commit intopython:mainfrom
vstinner:grp

Conversation

@vstinner
Copy link
Member

@vstinner vstinner commented Apr 17, 2024

@vstinner
Copy link
Member Author

I convert this PR to a draft until the main branch becomes Python 3.14.

@vstinner vstinner marked this pull request as ready for review May 8, 2024 16:27
@vstinner vstinner marked this pull request as draft May 8, 2024 16:34
@nineteendo
Copy link
Contributor

nineteendo commented May 9, 2024

Do we need to change the deprecation version to 3.16 now? And does it need to be in the 3.14 news file?

@vstinner
Copy link
Member Author

Do we need to change the deprecation version to 3.16 now? And does it need to be in the 3.14 news file?

Done, I rebased my PR on the main branch which is now Python 3.14.

@vstinner vstinner marked this pull request as ready for review May 10, 2024 09:15
@vstinner
Copy link
Member Author

@erlend-aasland @corona10: Do you want to review this change?

@corona10
Copy link
Member

Not now but at Pittsburgh

Copy link
Member

@corona10 corona10 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

So deprecate keyword arguments to avoid METH_VARARGS in the future?

@corona10
Copy link
Member

If yes, I am not sure that it is a good approach since we require the user to change their use case to optimize our implementation.

For me, this is a better approach

  • Optimize METH_VARARGS to become faster.
  • Not to declare METH_VARARGS as possible for the new APIs.

@vstinner
Copy link
Member Author

Ok, I close my PR. Thanks for the review!

@vstinner vstinner closed this May 19, 2024
@vstinner vstinner deleted the grp branch May 19, 2024 01:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments

X Tutup