| FazBrowse GitHub Viewer | Trending | | Home |
| Tools: [Download Repo ZIP] [Original HTTPS Page] |
Sorry, something went wrong.
|
@birkenfeld: Hi, do you recall why you considered to deprecated PyArg_Parse() when you documented the bytes() type in 2007? |
Sorry, something went wrong.
|
It makes sense because METH_OLDARGS had been deprecated, and removed already. |
Sorry, something went wrong.
|
PyArg_Parse() is used with METH_O and can be used for parsing separate arguments in other functions (for example if argument can be parsed as "i" or "s"), because there is no other public API for some format units or it is less convenient. You can find many examples in the CPython code. I believe that it is a useful function and should remain. |
Sorry, something went wrong.
|
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.
|
I had read only the doc. If there are enough use case for PyArg_Parse, the doc should be updated and deprecation comment should be removed. |
Sorry, something went wrong.
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.
I rewrote PyArg_Parse() documentation. Would you mind to review the updated PR? |
Sorry, something went wrong.
|
Just to leave my requested feedback, I don't remember anything about this specific function :) |
Sorry, something went wrong.
| Back | FazBrowse Home | New Git URL |
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.
📚 Documentation preview 📚: https://cpython-previews--105394.org.readthedocs.build/