Remove unnecessary clone in Neg and Not for &Array

Author: jturner314

src/impl_ops.rs

@@ -293,15 +293,15 @@ mod arithmetic_ops {
     }
 
     impl<'a, A, S, D> Neg for &'a ArrayBase<S, D>
-        where A: Clone + Neg<Output=A>,
+        where &'a A: 'a + Neg<Output=A>,
               S: Data<Elem=A>,
               D: Dimension
     {
         type Output = Array<A, D>;
         /// Perform an elementwise negation of reference `self` and return the
         /// result as a new `Array`.
         fn neg(self) -> Array<A, D> {
-            self.to_owned().neg()
+            self.map(Neg::neg)
         }
     }
 
@@ -321,15 +321,15 @@ mod arithmetic_ops {
     }
 
     impl<'a, A, S, D> Not for &'a ArrayBase<S, D>
-        where A: Clone + Not<Output=A>,
+        where &'a A: 'a + Not<Output=A>,
               S: Data<Elem=A>,
               D: Dimension
     {
         type Output = Array<A, D>;
         /// Perform an elementwise unary not of reference `self` and return the
         /// result as a new `Array`.
         fn not(self) -> Array<A, D> {
-            self.to_owned().not()
+            self.map(Not::not)
         }
     }
 }

tests/oper.rs

@@ -31,9 +31,12 @@ fn test_oper(op: &str, a: &[f32], b: &[f32], c: &[f32])
     test_oper_arr(op, aa.clone(), bb.clone(), cc.clone());
 }
 
-fn test_oper_arr<A: NdFloat + fmt::Debug, D: ndarray::Dimension>
-    (op: &str, mut aa: RcArray<A,D>, bb: RcArray<A, D>, cc: RcArray<A, D>)
-{
+fn test_oper_arr<D: Dimension>(
+    op: &str,
+    mut aa: RcArray<f32, D>,
+    bb: RcArray<f32, D>,
+    cc: RcArray<f32, D>,
+) {
     match op {
         "+" => {
             assert_eq!(&aa + &bb, cc);