Skip to content

More slot impl #6618

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 7 commits into from
Jan 1, 2026
Merged

More slot impl #6618

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
135e023
fix update_sub_slot
youknowone Dec 31, 2025
b2a3215
Fix __setitem__ __delitem__ handling
youknowone Dec 31, 2025
59cab45
contains
youknowone Dec 31, 2025
5d24adb
no __bool__ anymore
youknowone Jan 1, 2026
5871c72
Add rops to wrapper
youknowone Jan 1, 2026
9ae607f
Remove pymethod from set
youknowone Jan 1, 2026
bb18d0c
remove len
youknowone Jan 1, 2026
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
Next Next commit
fix update_sub_slot
  • Loading branch information
@youknowone
youknowone committed Jan 1, 2026
commit 135e02318b6f954e6efb7b59185a526fa766536e
10 changes: 8 additions & 2 deletions crates/vm/src/types/slot.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -607,7 +607,9 @@ impl PyType {
debug_assert!(name.as_str().ends_with("__"));

// Find all slot_defs matching this name and update each
for def in find_slot_defs_by_name(name.as_str()) {
// NOTE: Collect into Vec first to avoid issues during iteration
let defs: Vec<_> = find_slot_defs_by_name(name.as_str()).collect();
for def in defs {
self.update_one_slot::<ADD>(&def.accessor, name, ctx);
}

Expand Down Expand Up @@ -676,7 +678,11 @@ impl PyType {
macro_rules! update_sub_slot {
($group:ident, $slot:ident, $wrapper:expr, $variant:ident) => {{
if ADD {
if let Some(func) = self.lookup_slot_in_mro(name, ctx, |sf| {
// If this type defines the method itself (not inherited), use wrapper
// to ensure the Python method is called
if self.attributes.read().contains_key(name) {
self.slots.$group.$slot.store(Some($wrapper));
} else if let Some(func) = self.lookup_slot_in_mro(name, ctx, |sf| {
if let SlotFunc::$variant(f) = sf {
Some(*f)
} else {
Expand Down