Restore missing PyRef

philippeitis · web-flow · commit 633fc92fffde · 2020-02-27T14:12:18.000-08:00
diff --git a/vm/src/obj/objlist.rs b/vm/src/obj/objlist.rs
@@ -486,13 +486,13 @@ impl PyList {
     }
 
     #[pymethod(name = "__imul__")]
-    fn imul(zelf: PyRef<Self>, counter: isize, vm: &VirtualMachine) -> PyResult<Self> {
+    fn imul(zelf: PyRef<Self>, counter: isize, vm: &VirtualMachine) -> PyResult<PyRef<Self>> {
         if counter < 0 || zelf.len() * (counter as usize) < MAX_MEMORY_SIZE {
             let new_elements = sequence::seq_mul(&zelf.borrow_sequence(), counter)
                 .cloned()
                 .collect();
             zelf.elements.replace(new_elements);
-            return Ok(*zelf);
+            return Ok(zelf);
         }
         return Err(vm.new_memory_error("".to_owned()));
     }

Original file line number	Diff line number	Diff line change
`@@ -486,13 +486,13 @@ impl PyList {`
`486`	`486`	`}`
`487`	`487`
`488`	`488`	`#[pymethod(name = "__imul__")]`
`489`		`- fn imul(zelf: PyRef<Self>, counter: isize, vm: &VirtualMachine) -> PyResult<Self> {`
	`489`	`+ fn imul(zelf: PyRef<Self>, counter: isize, vm: &VirtualMachine) -> PyResult<PyRef<Self>> {`
`490`	`490`	`if counter < 0 \|\| zelf.len() * (counter as usize) < MAX_MEMORY_SIZE {`
`491`	`491`	`let new_elements = sequence::seq_mul(&zelf.borrow_sequence(), counter)`
`492`	`492`	`.cloned()`
`493`	`493`	`.collect();`
`494`	`494`	`zelf.elements.replace(new_elements);`
`495`		`- return Ok(*zelf);`
	`495`	`+ return Ok(zelf);`
`496`	`496`	`}`
`497`	`497`	`return Err(vm.new_memory_error("".to_owned()));`
`498`	`498`	`}`