Address comments

PyO3 · Feb 29, 2020 · f980d2c · f980d2c
1 parent 3a68f45
commit f980d2c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 10 deletions.
diff --git a/src/types/list.rs b/src/types/list.rs
@@ -183,17 +183,10 @@ macro_rules! array_impls {
         $(
             impl<T> IntoPy<PyObject> for [T; $N]
             where
-                T: Copy + IntoPy<PyObject>,
+                T: ToPyObject
             {
                 fn into_py(self, py: Python) -> PyObject {
-                    unsafe {
-                        let ptr = ffi::PyList_New(self.len() as Py_ssize_t);
-                        for (i, e) in self.iter().copied().enumerate() {
-                            let obj = e.into_py(py).into_ptr();
-                            ffi::PyList_SetItem(ptr, i as Py_ssize_t, obj);
-                        }
-                        PyObject::from_owned_ptr_or_panic(py, ptr)
-                    }
+                    self.as_ref().to_object(py)
                 }
             }
         )+

diff --git a/src/types/sequence.rs b/src/types/sequence.rs
@@ -330,7 +330,7 @@ where
     T: FromPyObject<'s>,
 {
     let seq = <PySequence as PyTryFrom>::try_from(obj)?;
-    if (seq.len()? as usize) != slice.len() {
+    if seq.len()? as usize != slice.len() {
         return Err(exceptions::BufferError::py_err(
             "Slice length does not match buffer length.",
         ));