Better documentation and iteration interface for value (#132)

mitsuhiko · Oct 23, 2022 · 34870b6 · 34870b6
1 parent 491ce5e
commit 34870b6
Show file tree

Hide file tree

Showing 5 changed files with 228 additions and 90 deletions.
diff --git a/minijinja/src/filters.rs b/minijinja/src/filters.rs
@@ -636,7 +636,7 @@ mod builtins {
         if count == 0 {
             return Err(Error::new(ErrorKind::InvalidOperation, "count cannot be 0"));
         }
-        let items = ok!(value.try_iter()).collect::<Vec<_>>();
+        let items = ok!(value.try_iter_owned()).collect::<Vec<_>>();
         let len = items.len();
         let items_per_slice = len / count;
         let slices_with_extra = len % count;
@@ -688,7 +688,7 @@ mod builtins {
         let mut rv = Vec::new();
         let mut tmp = Vec::with_capacity(count);
 
-        for item in ok!(value.try_iter()) {
+        for item in ok!(value.try_iter_owned()) {
             if tmp.len() == count {
                 rv.push(Value::from(mem::replace(
                     &mut tmp,

diff --git a/minijinja/src/key/mod.rs b/minijinja/src/key/mod.rs
@@ -269,7 +269,7 @@ pub mod key_interning {
 
         let v = Value::from_serializable(&vec![m.clone(), m.clone(), m.clone()]);
 
-        for value in v.try_iter().unwrap() {
+        for value in v.try_iter_owned().unwrap() {
             match value.0 {
                 ValueRepr::Map(m, _) => {
                     let k = m.iter().next().unwrap().0;