add_slot_wrappers must be called up to once

youknowone · youknowone · commit 4c9fa5a52127 · 2021-08-14T17:57:02.000+09:00
diff --git a/vm/src/pyobject.rs b/vm/src/pyobject.rs
@@ -389,11 +389,9 @@ impl PyContext {
     }
 
     pub fn add_slot_wrappers(&self, ty: &PyTypeRef) {
-        if !ty.attributes.read().contains_key("__new__") {
-            let new_wrapper =
-                self.new_bound_method(self.tp_new_wrapper.clone(), ty.clone().into_object());
-            ty.set_str_attr("__new__", new_wrapper);
-        }
+        let new_wrapper =
+            self.new_bound_method(self.tp_new_wrapper.clone(), ty.clone().into_object());
+        ty.set_str_attr("__new__", new_wrapper);
     }
 
     pub fn is_tp_new_wrapper(&self, obj: &PyObjectRef) -> bool {

Original file line number	Diff line number	Diff line change
`@@ -389,11 +389,9 @@ impl PyContext {`
`389`	`389`	`}`
`390`	`390`
`391`	`391`	`pub fn add_slot_wrappers(&self, ty: &PyTypeRef) {`
`392`		`- if !ty.attributes.read().contains_key("__new__") {`
`393`		`- let new_wrapper =`
`394`		`- self.new_bound_method(self.tp_new_wrapper.clone(), ty.clone().into_object());`
`395`		`- ty.set_str_attr("__new__", new_wrapper);`
`396`		`- }`
	`392`	`+ let new_wrapper =`
	`393`	`+ self.new_bound_method(self.tp_new_wrapper.clone(), ty.clone().into_object());`
	`394`	`+ ty.set_str_attr("__new__", new_wrapper);`
`397`	`395`	`}`
`398`	`396`
`399`	`397`	`pub fn is_tp_new_wrapper(&self, obj: &PyObjectRef) -> bool {`