Skip to content

Fix del obj.__dict__ to match CPython behavior (fixes #5355) #7568

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

Open
alok-108 wants to merge 13 commits into
base: main
Choose a base branch
from
Open

Fix del obj.__dict__ to match CPython behavior (fixes #5355) #7568

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
f1d9358
Fix del obj.__dict__ to match CPython behavior (issue #5355)
alok-108 Apr 6, 2026
28c5fe2
Address CodeRabbit concerns: fix GC clearing and improve thread safet…
alok-108 Apr 6, 2026
d3cc9d4
Fix del obj.__dict__: improve GC safety, implement lazy re-creation i…
alok-108 Apr 7, 2026
0b085ee
Restore expectedFailure for test_has_inline_values
alok-108 Apr 7, 2026
477c901
Fix ObjExt::new call site to include has_dict parameter
alok-108 Apr 7, 2026
4cc6d83
Remove stray test.py to avoid CI syntax errors
alok-108 Apr 7, 2026
f7f9e28
Remove debug txt files and clean test_class.py comments
alok-108 Apr 7, 2026
6ca9631
Delete Lib/test/test_class.py
alok-108 Apr 7, 2026
2564ed1
Restore test_class.py with correct changes (remove expectedFailure, n…
alok-108 Apr 7, 2026
7651712
Fix clippy warnings: remove unused into_inner, collapse nested if-let
alok-108 Apr 7, 2026
7882153
Fix rustfmt formatting and ruff PEP 8 E302 blank line
alok-108 Apr 8, 2026
143a92c
Align __dict__ error messages and ensure safety for function/partial …
alok-108 Apr 8, 2026
c46e482
Fix compilation errors: change &self to &Py<Self> in __dict__ methods
alok-108 Apr 8, 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
Prev Previous commit
Next Next commit
Fix rustfmt formatting and ruff PEP 8 E302 blank line
  • Loading branch information
@alok-108
alok-108 committed Apr 8, 2026
commit 7882153ab39e9737fded3c3d0ffbb3f6e3e331cd
6 changes: 1 addition & 5 deletions crates/vm/src/builtins/type.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2622,11 +2622,7 @@ fn subtype_get_dict(obj: PyObjectRef, vm: &VirtualMachine) -> PyResult {
}

// = subtype_setdict
fn subtype_set_dict(
obj: PyObjectRef,
value: PySetterValue,
vm: &VirtualMachine,
) -> PyResult<()> {
fn subtype_set_dict(obj: PyObjectRef, value: PySetterValue, vm: &VirtualMachine) -> PyResult<()> {
let base = get_builtin_base_with_dict(obj.class(), vm);

if let Some(base_type) = base {
Expand Down
5 changes: 4 additions & 1 deletion crates/vm/src/object/core.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -1080,7 +1080,10 @@ impl<T: PyPayload + core::fmt::Debug> PyInner<T> {
unsafe {
if let Some(offset) = ext_start {
let ext_ptr = alloc_ptr.add(offset) as *mut ObjExt;
let has_dict = typ.slots.flags.has_feature(crate::types::PyTypeFlags::HAS_DICT);
let has_dict = typ
.slots
.flags
.has_feature(crate::types::PyTypeFlags::HAS_DICT);
ext_ptr.write(ObjExt::new(dict, member_count, has_dict));
}

Expand Down
4 changes: 3 additions & 1 deletion crates/vm/src/protocol/object.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -201,7 +201,9 @@ impl PyObject {

if let Some(instance_dict) = self.instance_dict() {
if let PySetterValue::Assign(value) = value {
instance_dict.get_or_insert(vm).set_item(attr_name, value, vm)?;
instance_dict
.get_or_insert(vm)
.set_item(attr_name, value, vm)?;
} else if let Some(dict) = instance_dict.get() {
dict.del_item(attr_name, vm).map_err(|e| {
if e.fast_isinstance(vm.ctx.exceptions.key_error) {
Expand Down
1 change: 1 addition & 0 deletions extra_tests/snippets/test_del_dict.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
class C:
pass


obj = C()
obj.x = 42

Expand Down
Loading