Skip to content

bpo-32436: Implement PEP 567 #5027

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
1st1 merged 24 commits into from
Jan 23, 2018
Merged

bpo-32436: Implement PEP 567 #5027

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit Hold shift + click to select a range
8192f9a
bpo-32436: Implement PEP 567
Dec 28, 2017
8d0d64c
Add Token.MISSING, token.old_val; token.var
Dec 28, 2017
d846b41
Make PyContextVar_Get return a new ref
Dec 28, 2017
339c283
Explain HAMT
Dec 28, 2017
243175f
Mention how to debug HAMT
Dec 30, 2017
63c38fe
Address review feedback
Jan 15, 2018
c4750a7
ThreadState now refs to a Context object (fixes Context.run/__getitem__)
Jan 15, 2018
b1c97f0
Document hamt_t enum types
Jan 16, 2018
4f10c5e
Add a test
Jan 16, 2018
47d5a99
Add Context.copy() method
Jan 16, 2018
7549c6a
Harden PyContext_Exit to check that the current context matches
Jan 16, 2018
db384f3
Simplify context_new() logic
Jan 16, 2018
e299b4c
Token.reset() checks the context too now
Jan 16, 2018
dad0651
Token.old_val -> Token.old_value
Jan 16, 2018
78b0361
Test that subclassing is prohibited
Jan 17, 2018
3673bd9
Make ContextVar.reset raise an error for already used tokens
Jan 17, 2018
9223883
Drop unused file
Jan 17, 2018
8c82581
Reword a comment
Jan 17, 2018
17fa9c0
Address Inada-sun comments
Jan 18, 2018
52dc25b
Cache empty HAMT node
Jan 19, 2018
81ae871
Don't GC-track ContextVars if name/default are not tracked
Jan 19, 2018
7070cca
Handle an error if rehashing fails
Jan 19, 2018
cc8873c
PyContext_CopyCurrent() & PyContext_Copy() (sync with the PEP)
Jan 21, 2018
e353e95
Fix asynciomodule.c to use new PyContext_CopyCurrent
Jan 22, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Cache empty HAMT node
  • Loading branch information
Yury Selivanov
Yury Selivanov committed Jan 22, 2018
commit 52dc25ba70ddf194e5e05ac4f98aca0f5a2a8fd2
1 change: 1 addition & 0 deletions Lib/test/test_context.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -977,6 +977,7 @@ def test_hamt_gc_1(self):
A = HashKey(100, 'A')

h = hamt()
h = h.set(0, 0) # empty HAMT node is memoized in hamt.c
ref = weakref.ref(h)

a = []
Expand Down
15 changes: 15 additions & 0 deletions Python/hamt.c
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -337,6 +337,7 @@ typedef struct {


static PyHamtNode_Bitmap *_empty_bitmap_node;
static PyHamtObject *_empty_hamt;


static PyHamtObject *
Expand Down Expand Up @@ -2467,6 +2468,13 @@ hamt_alloc(void)
PyHamtObject *
_PyHamt_New(void)
{
if (_empty_hamt != NULL) {
/* HAMT is an immutable object so we can easily cache an
empty instance. */
Py_INCREF(_empty_hamt);
return _empty_hamt;
}

PyHamtObject *o = hamt_alloc();
if (o == NULL) {
return NULL;
Expand All @@ -2479,6 +2487,12 @@ _PyHamt_New(void)
}

o->h_count = 0;

if (_empty_hamt == NULL) {
Py_INCREF(o);
_empty_hamt = o;
}

return o;
}

Expand Down Expand Up @@ -2958,5 +2972,6 @@ _PyHamt_Init(void)
void
_PyHamt_Fini(void)
{
Py_CLEAR(_empty_hamt);
Py_CLEAR(_empty_bitmap_node);
}