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 — PEP 737: gh-111696: Add %T format to PyUnicode_FromFormat() by vstinner · Pull Request #111703 · python/cpython Conversation
|
|
I updated my PR:
|
Sorry, something went wrong.
There was an error while loading. Please reload this page.
50923bf to
6c09d87
Compare
6c09d87 to
86da2a3
Compare
|
I reduced this PR to the bare minimum, non controversial and most important: only add %T format just to add a type name ( Later, we can extend the API:
Extended the API later is not incompatible with this change: they would be additions. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
@serhiy-storchaka: Are you ok with this minor addition to PyUnicode_FromFormat()? My work on the limited C API is blocked by this missing feature. As I wrote previously, the format can be extended later if someone wants to cover most cases, add an alternative format, etc. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
I would prefer to make the size modifier mandatory and use it as the format specifier in It is also less ambiguous. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
My previous attempt in 2018: #9122 |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Previous attempt, merged commit 886483e, which was reverted a few days later. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
I wrote PEP 737 – Unify type name formatting for these changes: see the PEP discussion. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
PEP 737 changed the API since this PR was created. I close this PR for now and will create a new one (or maybe reopen this PR) since PEP 737 will be approved. |
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.