GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | |||||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | |||||||
GeezerVison is showing github.com — gh-105373: Remove PyArg_Parse() deprecation by vstinner · Pull Request #105394 · python/cpython Conversation
|
|
@birkenfeld: Hi, do you recall why you considered to deprecated PyArg_Parse() when you documented the |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
It makes sense because |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
PyArg_Parse() is used with I believe that it is a useful function and should remain. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Argument Clinic generates PyArg_Parse() calls for some function signatures. There are around 69 calls to PyArg_Parse() in the CPython code base, included 15 genereated by Argument Clinic. If we really want to get ride of it, we should be good citizen and start by cleaning our own code base. But here I agree with @serhiy-storchaka, this function is fine and should be kept. If there are use cases where others functions are more appropriated, maybe the doc should be enhanced? @methane: What do you think? |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
I had read only the doc. If there are enough use case for |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There is no plan to deprecate PyArg_Parse(). The deprecation was added as a comment in the C API documentation in 2007 by commit 85eb8c1.
a4e9a4d to
ecee586
Compare
I rewrote PyArg_Parse() documentation. Would you mind to review the updated PR? |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Just to leave my requested feedback, I don't remember anything about this specific function :) |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
There was an error while loading. Please reload this page.