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 — [WIP] gh-141510: PEP 814: Add built-in frozendict type by vstinner · Pull Request #141508 · python/cpython Conversation
|
|
This PR was created mainly to play with PEP 814 implementation, not to be merged. If PEP 814 is accepted, this PR should be splitted into smaller parts, easier to review. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
mypy fails with |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Tests / Android (x86_64) (pull_request) does crash, I'm not sure why. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Co-authored-by: Nice Zombies <[email protected]>
It should be fixed by Add PyFrozenDict_Type to static_types commit. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
There was an error while loading. Please reload this page.
Co-authored-by: kalvdans <[email protected]>
|
TODO: Provide better error on hashing a mutable value: |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Add GET_USED() macro.
There was an error while loading. Please reload this page.
|
This PR is a draft combining all changes to adopt frozendict in the stdlib. It's too big to be merged. I started to split this PR into smaller PRs with #144757 as a starting point: most basic PR just to add the frozendict type. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
@vstinner is it purposed that this draft PR is linked in the documentation at „reference implementation“ in PEP 814? I know that this includes more than the other PR, but I think that it was probably better to link the real PRs with the real changes made because they include really every single change correctly rather than a draft, no? |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
@benediktjohannes PEPs are historical documents (PEP 814 will soon have a banner to this effect), so it's fine for https://peps.python.org/pep-0814/#reference-implementation to point to this PR. It was the reference implementation at the time the PEP was written, discussed and accepted. The real implementation is another matter, and is being committed in smaller PRs. We can follow along via the issue, #141510. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Thanks! |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
I created many PRs in #141510. This PR is no longer needed, I close it. |
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.