Skip to content

Implement Mapping Protocol #3121

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
youknowone merged 14 commits into from
Oct 2, 2021
Merged

Implement Mapping Protocol #3121

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
14ce6aa
add slot_as_mapping slots with atomic cell
Snowapril Sep 20, 2021
77351b3
add PyMapping struct for mapping protocol
Snowapril Sep 20, 2021
1b6cdf5
rename length to compute_length
Snowapril Sep 20, 2021
fc761b4
add mapping protocol to proper types
Snowapril Sep 20, 2021
e83b68b
apply mapping protocol in itemprotocol
Snowapril Sep 20, 2021
b197a17
switch RangeIndex to PyObjectRef
Snowapril Sep 22, 2021
4f009ef
switch SequenceIndex to PyObjectRef
Snowapril Sep 22, 2021
1fbbdaa
support delitem in ass_subscript
Snowapril Sep 22, 2021
6e11b89
add mapping protocol methods to update_slot
Snowapril Sep 29, 2021
9e5120d
add mapping type check to mappingproxy
Snowapril Sep 29, 2021
4fd0681
move mapping protocol to rustpython_vm::protocol
Snowapril Sep 29, 2021
aa2a7f0
add PyMapping as PyObjectRef wrapper
Snowapril Sep 30, 2021
a81b3f9
fix posix & winapi to use proper PyMapping
Snowapril Oct 2, 2021
7b0965c
turn unreachable function to cold
youknowone Oct 2, 2021
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
turn unreachable function to cold
  • Loading branch information
@youknowone
youknowone authored Oct 2, 2021
commit 7b0965c1d77e0f4b82c2ef128832dd1d1f6eb947
2 changes: 1 addition & 1 deletion vm/src/builtins/bytes.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -559,7 +559,7 @@ impl AsMapping for PyBytes {
Self::downcast_ref(&zelf, vm).map(|zelf| zelf.getitem(needle, vm))?
}

#[inline]
#[cold]
fn ass_subscript(
zelf: PyObjectRef,
_needle: PyObjectRef,
Expand Down
2 changes: 1 addition & 1 deletion vm/src/builtins/mappingproxy.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -158,7 +158,7 @@ impl AsMapping for PyMappingProxy {
Self::downcast_ref(&zelf, vm).map(|zelf| zelf.getitem(needle, vm))?
}

#[inline]
#[cold]
fn ass_subscript(
zelf: PyObjectRef,
_needle: PyObjectRef,
Expand Down