Correct PyInner<T> fields order

moreal · youknowone · commit e27538f9428c · 2022-09-18T06:49:37.000+09:00
Because `PyInner&lt;T&gt;` struct follows C langauge's representation,
when passing `impl From&lt;PyRef&lt;T&gt;&gt; for PyObjectRef`,
the unsafe function,  `NonNull::cast`, harms the `slots` fields.

This reorders the fields to make `payload` field go to the last.
diff --git a/vm/src/object/core.rs b/vm/src/object/core.rs
@@ -114,9 +114,9 @@ struct PyInner<T> {
     typ: PyRwLock<PyTypeRef>, // __class__ member
     dict: Option<InstanceDict>,
     weak_list: WeakRefList,
+    slots: Box<[PyRwLock<Option<PyObjectRef>>]>,
 
     payload: T,
-    slots: Box<[PyRwLock<Option<PyObjectRef>>]>,
 }
 
 impl<T: fmt::Debug> fmt::Debug for PyInner<T> {

Original file line number	Diff line number	Diff line change
`@@ -114,9 +114,9 @@ struct PyInner<T> {`
`114`	`114`	`typ: PyRwLock<PyTypeRef>, // __class__ member`
`115`	`115`	`dict: Option<InstanceDict>,`
`116`	`116`	`weak_list: WeakRefList,`
	`117`	`+ slots: Box<[PyRwLock<Option<PyObjectRef>>]>,`
`117`	`118`
`118`	`119`	`payload: T,`
`119`		`- slots: Box<[PyRwLock<Option<PyObjectRef>>]>,`
`120`	`120`	`}`
`121`	`121`
`122`	`122`	`impl<T: fmt::Debug> fmt::Debug for PyInner<T> {`