From e7ca6d6a3d05bb770017d87ae25a89ae305600c5 Mon Sep 17 00:00:00 2001
From: Jim Turner <github@turner.link>
Date: Thu, 18 Feb 2021 14:07:02 -0500
Subject: [PATCH] Fix clippy warnings

---
 src/data_traits.rs                | 2 ++
 src/impl_internal_constructors.rs | 4 ++--
 src/linalg/impl_linalg.rs         | 1 +
 3 files changed, 5 insertions(+), 2 deletions(-)

diff --git a/src/data_traits.rs b/src/data_traits.rs
index 2b59fcb8d..1e191c468 100644
--- a/src/data_traits.rs
+++ b/src/data_traits.rs
@@ -94,6 +94,7 @@ pub unsafe trait RawDataClone: RawData {
 pub unsafe trait Data: RawData {
     /// Converts the array to a uniquely owned array, cloning elements if necessary.
     #[doc(hidden)]
+    #[allow(clippy::wrong_self_convention)]
     fn into_owned<D>(self_: ArrayBase<Self, D>) -> ArrayBase<OwnedRepr<Self::Elem>, D>
     where
         Self::Elem: Clone,
@@ -102,6 +103,7 @@ pub unsafe trait Data: RawData {
     /// Return a shared ownership (copy on write) array based on the existing one,
     /// cloning elements if necessary.
     #[doc(hidden)]
+    #[allow(clippy::wrong_self_convention)]
     fn to_shared<D>(self_: &ArrayBase<Self, D>) -> ArrayBase<OwnedArcRepr<Self::Elem>, D>
     where
         Self::Elem: Clone,
diff --git a/src/impl_internal_constructors.rs b/src/impl_internal_constructors.rs
index 0ed60a622..5d47c9897 100644
--- a/src/impl_internal_constructors.rs
+++ b/src/impl_internal_constructors.rs
@@ -25,8 +25,8 @@ where
     /// See ArrayView::from_shape_ptr for general pointer validity documentation.
     pub(crate) unsafe fn from_data_ptr(data: S, ptr: NonNull<A>) -> Self {
         let array = ArrayBase {
-            data: data,
-            ptr: ptr,
+            data,
+            ptr,
             dim: Ix1(0),
             strides: Ix1(1),
         };
diff --git a/src/linalg/impl_linalg.rs b/src/linalg/impl_linalg.rs
index 878444c26..1cc99c59e 100644
--- a/src/linalg/impl_linalg.rs
+++ b/src/linalg/impl_linalg.rs
@@ -613,6 +613,7 @@ pub fn general_mat_vec_mul<A, S1, S2, S3>(
 ///
 /// The caller must ensure that the raw view is valid for writing.
 /// the destination may be uninitialized iff beta is zero.
+#[allow(clippy::collapsible_if)]
 unsafe fn general_mat_vec_mul_impl<A, S1, S2>(
     alpha: A,
     a: &ArrayBase<S1, Ix2>,