Revert changes on NodeSeq because I'm not sure it's useful at all

Author: cecton

packages/yew/src/utils/mod.rs

@@ -2,8 +2,6 @@
 
 use std::marker::PhantomData;
 
-use implicit_clone::unsync::IArray;
-use implicit_clone::ImplicitClone;
 use yew::html::ChildrenRenderer;
 
 /// Map `IntoIterator<Item = Into<T>>` to `Iterator<Item = T>`
@@ -17,64 +15,38 @@ where
 
 /// A special type necessary for flattening components returned from nested html macros.
 #[derive(Debug)]
-pub struct NodeSeq<IN, OUT: ImplicitClone + 'static>(IArray<OUT>, PhantomData<IN>);
+pub struct NodeSeq<IN, OUT>(Vec<OUT>, PhantomData<IN>);
 
-impl<IN: Into<OUT>, OUT: ImplicitClone + 'static> From<IN> for NodeSeq<IN, OUT> {
+impl<IN: Into<OUT>, OUT> From<IN> for NodeSeq<IN, OUT> {
     fn from(val: IN) -> Self {
-        Self(IArray::Single([val.into()]), PhantomData)
+        Self(vec![val.into()], PhantomData)
     }
 }
 
-impl<IN: Into<OUT>, OUT: ImplicitClone + 'static> From<Option<IN>> for NodeSeq<IN, OUT> {
+impl<IN: Into<OUT>, OUT> From<Option<IN>> for NodeSeq<IN, OUT> {
     fn from(val: Option<IN>) -> Self {
-        Self(
-            val.map(|s| IArray::Single([s.into()])).unwrap_or_default(),
-            PhantomData,
-        )
+        Self(val.map(|s| vec![s.into()]).unwrap_or_default(), PhantomData)
     }
 }
 
-impl<IN: Into<OUT>, OUT: ImplicitClone + 'static> From<Vec<IN>> for NodeSeq<IN, OUT> {
-    fn from(mut val: Vec<IN>) -> Self {
-        if val.len() == 1 {
-            let item = val.pop().unwrap();
-            Self(IArray::Single([item.into()]), PhantomData)
-        } else {
-            Self(val.into_iter().map(|x| x.into()).collect(), PhantomData)
-        }
+impl<IN: Into<OUT>, OUT> From<Vec<IN>> for NodeSeq<IN, OUT> {
+    fn from(val: Vec<IN>) -> Self {
+        Self(val.into_iter().map(|x| x.into()).collect(), PhantomData)
     }
 }
 
-impl<IN: Into<OUT> + ImplicitClone, OUT: ImplicitClone + 'static> From<IArray<IN>>
-    for NodeSeq<IN, OUT>
-{
-    fn from(val: IArray<IN>) -> Self {
-        Self(val.iter().map(|x| x.into()).collect(), PhantomData)
-    }
-}
-
-impl<IN: Into<OUT> + ImplicitClone, OUT: ImplicitClone + 'static> From<&IArray<IN>>
-    for NodeSeq<IN, OUT>
-{
-    fn from(val: &IArray<IN>) -> Self {
-        Self(val.iter().map(|x| x.into()).collect(), PhantomData)
-    }
-}
-
-impl<IN: Into<OUT> + Clone, OUT: ImplicitClone + 'static> From<&ChildrenRenderer<IN>>
-    for NodeSeq<IN, OUT>
-{
+impl<IN: Into<OUT> + Clone, OUT> From<&ChildrenRenderer<IN>> for NodeSeq<IN, OUT> {
     fn from(val: &ChildrenRenderer<IN>) -> Self {
         Self(val.iter().map(|x| x.into()).collect(), PhantomData)
     }
 }
 
-impl<IN, OUT: ImplicitClone + 'static> IntoIterator for NodeSeq<IN, OUT> {
-    type IntoIter = implicit_clone::unsync::Iter<Self::Item>;
+impl<IN, OUT> IntoIterator for NodeSeq<IN, OUT> {
+    type IntoIter = std::vec::IntoIter<Self::Item>;
     type Item = OUT;
 
     fn into_iter(self) -> Self::IntoIter {
-        self.0.iter()
+        self.0.into_iter()
     }
 }