luxolus
/
pyo3
2
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #358 from chr1sj0nes/pylist_reverse 

Add `PyList.reverse()`.
This commit is contained in:
Yuji Kanagawa 2019-02-18 11:36:07 +09:00 committed by GitHub
parent b01592e201 5a28a1c4f0
commit f248aaddbe
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 22 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
src/types/list.rs
View File

@ -125,6 +125,11 @@ impl PyList {
pub fn sort(&self) -> PyResult<()> { pub fn sort(&self) -> PyResult<()> {
unsafe { err::error_on_minusone(self.py(), ffi::PyList_Sort(self.as_ptr())) } unsafe { err::error_on_minusone(self.py(), ffi::PyList_Sort(self.as_ptr())) }
} }
/// Reverses the list in-place. Equivalent to python `l.reverse()`
pub fn reverse(&self) -> PyResult<()> {
unsafe { err::error_on_minusone(self.py(), ffi::PyList_Reverse(self.as_ptr())) }
}
} }
/// Used by `PyList::iter()`. /// Used by `PyList::iter()`.
@ -403,4 +408,21 @@ mod test {
assert_eq!(5, list.get_item(2).extract::<i32>().unwrap()); assert_eq!(5, list.get_item(2).extract::<i32>().unwrap());
assert_eq!(7, list.get_item(3).extract::<i32>().unwrap()); assert_eq!(7, list.get_item(3).extract::<i32>().unwrap());
} }
#[test]
fn test_reverse() {
let gil = Python::acquire_gil();
let py = gil.python();
let v = vec![2, 3, 5, 7];
let list = PyList::new(py, &v);
assert_eq!(2, list.get_item(0).extract::<i32>().unwrap());
assert_eq!(3, list.get_item(1).extract::<i32>().unwrap());
assert_eq!(5, list.get_item(2).extract::<i32>().unwrap());
assert_eq!(7, list.get_item(3).extract::<i32>().unwrap());
list.reverse().unwrap();
assert_eq!(7, list.get_item(0).extract::<i32>().unwrap());
assert_eq!(5, list.get_item(1).extract::<i32>().unwrap());
assert_eq!(3, list.get_item(2).extract::<i32>().unwrap());
assert_eq!(2, list.get_item(3).extract::<i32>().unwrap());
}
} }