From 612b60fade156995ec8f3aac5fe93f5cf09b1f6e Mon Sep 17 00:00:00 2001
From: Mara Bos <m-ou.se@m-ou.se>
Date: Fri, 29 Nov 2019 15:31:52 +0100
Subject: [PATCH] Implement Extend<&N> for VecStorage.

Extend<N> was already implemented, but nalgebra vectors/matrices give
iterators that give &N, not N, so implementing Extend<&N> as well makes
it easier to use.

It seems common practice to do so: The standard library's Vec also
implments Extend for both T and &T.
---
 src/base/vec_storage.rs | 15 +++++++++++++++
 1 file changed, 15 insertions(+)

diff --git a/src/base/vec_storage.rs b/src/base/vec_storage.rs
index a0230488..747e5179 100644
--- a/src/base/vec_storage.rs
+++ b/src/base/vec_storage.rs
@@ -268,6 +268,21 @@ impl<N, R: Dim> Extend<N> for VecStorage<N, R, Dynamic>
     }
 }
 
+impl<'a, N: 'a + Copy, R: Dim> Extend<&'a N> for VecStorage<N, R, Dynamic>
+{
+    /// Extends the number of columns of the `VecStorage` with elements
+    /// from the given iterator.
+    ///
+    /// # Panics
+    /// This function panics if the number of elements yielded by the
+    /// given iterator is not a multiple of the number of rows of the
+    /// `VecStorage`.
+    fn extend<I: IntoIterator<Item=&'a N>>(&mut self, iter: I)
+    {
+        self.extend(iter.into_iter().copied())
+    }
+}
+
 impl<N, R, RV, SV> Extend<Vector<N, RV, SV>> for VecStorage<N, R, Dynamic>
 where
     N: Scalar + Copy,