Add index check

yt2b · yt2b · commit 62d071f08878 · 2022-12-11T00:48:34.000+09:00
diff --git a/vm/src/builtins/list.rs b/vm/src/builtins/list.rs
@@ -300,7 +300,8 @@ impl PyList {
 
         if let Some(index) = index.into() {
             // defer delete out of borrow
-            Ok(self.borrow_vec_mut().remove(index))
+            let mut elements = self.borrow_vec_mut();
+            Ok((index < elements.len()).then(move || elements.remove(index)))
         } else {
             Err(vm.new_value_error(format!("'{}' is not in list", needle.str(vm)?)))
         }

Original file line number	Diff line number	Diff line change
`@@ -300,7 +300,8 @@ impl PyList {`
`300`	`300`
`301`	`301`	`if let Some(index) = index.into() {`
`302`	`302`	`// defer delete out of borrow`
`303`		`- Ok(self.borrow_vec_mut().remove(index))`
	`303`	`+ let mut elements = self.borrow_vec_mut();`
	`304`	`+ Ok((index < elements.len()).then(move \|\| elements.remove(index)))`
`304`	`305`	`} else {`
`305`	`306`	`Err(vm.new_value_error(format!("'{}' is not in list", needle.str(vm)?)))`
`306`	`307`	`}`