RustPython
diff --git a/‎vm/src/stdlib/ctypes/array.rs
Copy file name to clipboardExpand all lines: vm/src/stdlib/ctypes/array.rs
+6-5Lines changed: 6 additions & 5 deletions b/‎vm/src/stdlib/ctypes/array.rs
Copy file name to clipboardExpand all lines: vm/src/stdlib/ctypes/array.rs
+6-5Lines changed: 6 additions & 5 deletions
diff --git a/‎vm/src/stdlib/ctypes/basics.rs
Copy file name to clipboardExpand all lines: vm/src/stdlib/ctypes/basics.rs
+5-3Lines changed: 5 additions & 3 deletions b/‎vm/src/stdlib/ctypes/basics.rs
Copy file name to clipboardExpand all lines: vm/src/stdlib/ctypes/basics.rs
+5-3Lines changed: 5 additions & 3 deletions
diff --git a/‎vm/src/stdlib/ctypes/primitive.rs
Copy file name to clipboardExpand all lines: vm/src/stdlib/ctypes/primitive.rs
+4-2Lines changed: 4 additions & 2 deletions b/‎vm/src/stdlib/ctypes/primitive.rs
Copy file name to clipboardExpand all lines: vm/src/stdlib/ctypes/primitive.rs
+4-2Lines changed: 4 additions & 2 deletions
@@ -3,7 +3,6 @@ use super::{
     primitive::{new_simple_type, PyCSimple},
 };
 use crate::builtins::{
-    self,
     slice::PySlice,
     PyBytes, PyInt, PyList, PyRange, PyStr, PyType, PyTypeRef,
 };
@@ -14,7 +13,7 @@ use crate::stdlib::ctypes::basics::{
     default_from_param, generic_get_buffer, get_size, BorrowValue as BorrowValueCData,
     BorrowValueMut, PyCData, PyCDataFunctions, PyCDataMethods, PyCDataSequenceMethods, RawBuffer,
 };
-use crate::{AsObject, Context, PyObjectRef, PyRef, PyResult, TryFromObject, VirtualMachine};
+use crate::{AsObject, Py, PyObjectRef, PyRef, PyResult, TryFromObject, VirtualMachine};
 use rustpython_vm::object::PyPayload;
 use num_traits::Signed;
 use std::convert::TryInto;
@@ -23,6 +22,7 @@ use widestring::WideCString;
 use crate::class::StaticType;
 use crate::convert::IntoObject;
 use crate::protocol::{PyBuffer, PyIter};
+use crate::types::AsBuffer;
 
 // TODO: make sure that this is correct wrt windows and unix wstr
 fn slice_to_obj(ty: &str, b: &[u8], vm: &VirtualMachine) -> PyResult {
@@ -256,7 +256,7 @@ fn array_slice_setitem(
     //Right now I'm setting one
     let size = length.map_or(Ok(1), |v| usize::try_from_object(vm, v))?;
 
-    for (i, curr) in PyIterable::try_from_object(vm,_range.into_object(vm))?.iter(vm)?.enumerate() {
+    for (i, curr) in PyIter::try_from_object(vm,_range.into_object(vm))?.iter(vm)?.enumerate() {
         let idx = fix_index(isize::try_from_object(vm, curr?)?, size, vm)? as usize;
         let offset = idx * size;
         let item = obj.get_item(i, vm)?;
@@ -286,6 +286,7 @@ fn fix_index(index: isize, length: usize, vm: &VirtualMachine) -> PyResult<isize
 #[pyclass(module = "_ctypes", name = "PyCArrayType", base = "PyType")]
 pub struct PyCArrayMeta {}
 
+#[derive(PyPayload)]
 #[pyclass(
     module = "_ctypes",
     name = "Array",
@@ -327,8 +328,8 @@ impl<'a> BorrowValueMut<'a> for PyCArray {
     }
 }
 
-impl BufferProtocol for PyCArray {
-    fn get_buffer(zelf: &PyRef<Self>, vm: &VirtualMachine) -> PyResult<Box<dyn Buffer>> {
+impl AsBuffer for PyCArray {
+    fn as_buffer(zelf: &Py<Self>, vm: &VirtualMachine) -> PyResult<PyBuffer> {
         generic_get_buffer::<Self>(zelf, vm)
     }
 }
 
@@ -17,6 +17,7 @@ use crate::function::Either;
 use crate::builtins::PyTypeRef;
 use crate::protocol::PyBuffer;
 use crossbeam_utils::atomic::AtomicCell;
+use crate::types::AsBuffer;
 
 pub fn get_size(ty: &str) -> usize {
     match ty {
@@ -270,7 +271,7 @@ pub trait PyCDataSequenceMethods: PyPayload {
     }
 }
 
-pub fn generic_get_buffer<T>(zelf: &PyRef<T>, vm: &VirtualMachine) -> PyResult<Box<dyn Buffer>>
+pub fn generic_get_buffer<T>(zelf: &Py<T>, vm: &VirtualMachine) -> PyResult<PyBuffer>
 where
         for<'a> T: PyPayload + fmt::Debug + BorrowValue<'a> + BorrowValueMut<'a>,
 {
@@ -312,8 +313,8 @@ impl<'a> BorrowValueMut<'a> for PyCData {
     }
 }
 
-impl BufferProtocol for PyCData {
-    fn get_buffer(zelf: &PyRef<Self>, vm: &VirtualMachine) -> PyResult<Box<dyn Buffer>> {
+impl AsBuffer for PyCData {
+    fn as_buffer(zelf: &PyRef<Self>, vm: &VirtualMachine) -> PyResult<PyBuffer> {
         generic_get_buffer::<Self>(zelf, vm)
     }
 }
@@ -371,6 +372,7 @@ unsafe impl Sync for RawBuffer {}
 // This Trait is the equivalent of PyCData_Type on tp_base for
 // Struct_Type, Union_Type, PyCPointer_Type
 // PyCArray_Type, PyCSimple_Type, PyCFuncPtr_Type
+#[derive(PyPayload)]
 #[pyclass(module = "_ctypes", name = "_CData")]
 pub struct PyCData {
     _objects: AtomicCell<Vec<PyObjectRef>>,
 
@@ -15,7 +15,7 @@ use crate::stdlib::ctypes::basics::{
 use crate::stdlib::ctypes::function::PyCFuncPtr;
 use crate::stdlib::ctypes::pointer::PyCPointer;
 use crate::function::Either;
-use crate::{PyObjectRef, PyRef, PyResult, VirtualMachine};
+use crate::{PyObjectRef, PyPayload, PyRef, PyResult, VirtualMachine};
 use crate::protocol::PyBuffer;
 
 const SIMPLE_TYPE_CHARS: &str = "cbBhHiIlLdfguzZPqQ?";
@@ -259,20 +259,22 @@ pub fn new_simple_type(
     }
 }
 
+#[derive(PyPayload)]
 #[pyclass(module = "_ctypes", name = "PyCSimpleType", base = "PyType")]
 pub struct PySimpleMeta {}
 
 #[pyclass(with(PyCDataMethods), flags(BASETYPE))]
 impl PySimpleMeta {
     #[pyslot]
-    fn tp_new(cls: PyTypeRef, _: OptionalArg, vm: &VirtualMachine) -> PyResult {
+    fn new(cls: PyTypeRef, _: OptionalArg, vm: &VirtualMachine) -> PyResult {
         Ok(new_simple_type(Either::B(&cls), vm)?
             .into_ref_with_type(vm, cls)?
             .as_object()
             .clone())
     }
 }
 
+#[derive(PyPayload)]
 #[pyclass(
     module = "_ctypes",
     name = "_SimpleCData",
Original file line number	Diff line number	Diff line change
`@@ -17,6 +17,7 @@ use crate::function::Either;`
`17`	`17`	`use crate::builtins::PyTypeRef;`
`18`	`18`	`use crate::protocol::PyBuffer;`
`19`	`19`	`use crossbeam_utils::atomic::AtomicCell;`
	`20`	`+use crate::types::AsBuffer;`
`20`	`21`
`21`	`22`	`pub fn get_size(ty: &str) -> usize {`
`22`	`23`	`match ty {`
`@@ -270,7 +271,7 @@ pub trait PyCDataSequenceMethods: PyPayload {`
`270`	`271`	`}`
`271`	`272`	`}`
`272`	`273`
`273`		`-pub fn generic_get_buffer<T>(zelf: &PyRef<T>, vm: &VirtualMachine) -> PyResult<Box<dyn Buffer>>`
	`274`	`+pub fn generic_get_buffer<T>(zelf: &Py<T>, vm: &VirtualMachine) -> PyResult<PyBuffer>`
`274`	`275`	`where`
`275`	`276`	`for<'a> T: PyPayload + fmt::Debug + BorrowValue<'a> + BorrowValueMut<'a>,`
`276`	`277`	`{`
`@@ -312,8 +313,8 @@ impl<'a> BorrowValueMut<'a> for PyCData {`
`312`	`313`	`}`
`313`	`314`	`}`
`314`	`315`
`315`		`-impl BufferProtocol for PyCData {`
`316`		`- fn get_buffer(zelf: &PyRef<Self>, vm: &VirtualMachine) -> PyResult<Box<dyn Buffer>> {`
	`316`	`+impl AsBuffer for PyCData {`
	`317`	`+ fn as_buffer(zelf: &PyRef<Self>, vm: &VirtualMachine) -> PyResult<PyBuffer> {`
`317`	`318`	`generic_get_buffer::<Self>(zelf, vm)`
`318`	`319`	`}`
`319`	`320`	`}`
`@@ -371,6 +372,7 @@ unsafe impl Sync for RawBuffer {}`
`371`	`372`	`// This Trait is the equivalent of PyCData_Type on tp_base for`
`372`	`373`	`// Struct_Type, Union_Type, PyCPointer_Type`
`373`	`374`	`// PyCArray_Type, PyCSimple_Type, PyCFuncPtr_Type`
	`375`	`+#[derive(PyPayload)]`
`374`	`376`	`#[pyclass(module = "_ctypes", name = "_CData")]`
`375`	`377`	`pub struct PyCData {`
`376`	`378`	`_objects: AtomicCell<Vec<PyObjectRef>>,`