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

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 clippy warnings: remove unused into_inner, collapse nested if-let
  • Loading branch information
@alok-108
alok-108 committed Apr 7, 2026
commit 7651712c7336d919d6b7312bd07fb74d41f28c4c
11 changes: 2 additions & 9 deletions crates/vm/src/object/core.rs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -954,11 +954,6 @@ impl InstanceDict {
core::mem::replace(&mut self.d.write(), d)
}

/// Consume the InstanceDict and return the inner Option<PyDictRef>.
#[inline]
pub fn into_inner(self) -> Option<PyDictRef> {
self.d.into_inner()
}

pub(crate) fn get_or_insert(&self, vm: &VirtualMachine) -> PyDictRef {
if let Some(existing) = self.d.read().as_ref() {
Expand Down Expand Up @@ -1800,10 +1795,8 @@ impl PyObject {
let ext_ptr =
core::ptr::with_exposed_provenance_mut::<ObjExt>(self_addr.wrapping_sub(offset));
let ext = unsafe { &mut *ext_ptr };
if let Some(instance_dict) = &ext.dict {
if let Some(dict_ref) = instance_dict.replace(None) {
result.push(dict_ref.into());
}
if let Some(dict_ref) = ext.dict.as_ref().and_then(|d| d.replace(None)) {
result.push(dict_ref.into());
}
for slot in ext.slots.iter() {
if let Some(val) = slot.write().take() {
Expand Down
Loading