list_list::Builder::set() takes an 'impl SetterInput'

Author: dwrensha

capnp/src/list_list.rs

@@ -23,7 +23,7 @@
 
 use crate::introspect;
 use crate::private::layout::{ListBuilder, ListReader, Pointer, PointerBuilder, PointerReader};
-use crate::traits::{FromPointerBuilder, FromPointerReader, IndexMove, ListIter};
+use crate::traits::{FromPointerBuilder, FromPointerReader, IndexMove, ListIter, SetterInput};
 use crate::Result;
 
 #[derive(Clone, Copy)]
@@ -239,16 +239,13 @@ where
         }
     }
 
-    pub fn set<'b>(&mut self, index: u32, value: T::Reader<'a>) -> Result<()>
-    where
-        T::Reader<'a>: crate::traits::IntoInternalListReader<'b>,
-    {
-        use crate::traits::IntoInternalListReader;
+    pub fn set(&mut self, index: u32, value: impl SetterInput<T>) -> Result<()> {
         assert!(index < self.len());
-        self.builder
-            .reborrow()
-            .get_pointer_element(index)
-            .set_list(&value.into_internal_list_reader(), false)
+        SetterInput::set_pointer_builder(
+            self.builder.reborrow().get_pointer_element(index),
+            value,
+            false,
+        )
     }
 }
 

capnpc/test/test.rs

@@ -370,11 +370,8 @@ mod tests {
             {
                 let mut prim_list_list = test_complex_list.reborrow().init_prim_list_list(2);
                 {
-                    let mut prim_list = prim_list_list.reborrow().init(0, 3);
-                    prim_list.set(0, 5);
-                    prim_list.set(1, 6);
-                    prim_list.set(2, 7);
-                    assert_eq!(prim_list.len(), 3);
+                    prim_list_list.set(0, &[5, 6, 7]).unwrap();
+                    assert_eq!(prim_list_list.reborrow().get(0).unwrap().len(), 3);
                 }
                 let mut prim_list = prim_list_list.init(1, 1);
                 prim_list.set(0, -1);