Merge pull request #1381 from rust-ndarray/inline

Use inline on spit_at and smaller methods

Author: bluss

src/impl_internal_constructors.rs

@@ -22,6 +22,7 @@ where S: RawData<Elem = A>
     /// The caller must ensure that the data storage and pointer is valid.
     ///
     /// See ArrayView::from_shape_ptr for general pointer validity documentation.
+    #[inline]
     pub(crate) unsafe fn from_data_ptr(data: S, ptr: NonNull<A>) -> Self
     {
         let array = ArrayBase {
@@ -50,6 +51,7 @@ where
     ///
     /// The caller needs to ensure that the new strides and dimensions are correct
     /// for the array data.
+    #[inline]
     pub(crate) unsafe fn with_strides_dim<E>(self, strides: E, dim: E) -> ArrayBase<S, E>
     where E: Dimension
     {

src/impl_raw_views.rs

@@ -21,6 +21,7 @@ where D: Dimension
         RawArrayView::from_data_ptr(RawViewRepr::new(), ptr).with_strides_dim(strides, dim)
     }
 
+    #[inline]
     unsafe fn new_(ptr: *const A, dim: D, strides: D) -> Self
     {
         Self::new(nonnull_debug_checked_from_ptr(ptr as *mut A), dim, strides)
@@ -66,6 +67,7 @@ where D: Dimension
     /// but it's not a complete check.
     ///
     /// [`.offset()`]: https://doc.rust-lang.org/stable/std/primitive.pointer.html#method.offset
+    #[inline]
     pub unsafe fn from_shape_ptr<Sh>(shape: Sh, ptr: *const A) -> Self
     where Sh: Into<StrideShape<D>>
     {
@@ -107,6 +109,7 @@ where D: Dimension
     ///
     /// **Panics** if `axis` or `index` is out of bounds.
     #[track_caller]
+    #[inline]
     pub fn split_at(self, axis: Axis, index: Ix) -> (Self, Self)
     {
         assert!(index <= self.len_of(axis));
@@ -234,6 +237,7 @@ where D: Dimension
         RawArrayViewMut::from_data_ptr(RawViewRepr::new(), ptr).with_strides_dim(strides, dim)
     }
 
+    #[inline]
     unsafe fn new_(ptr: *mut A, dim: D, strides: D) -> Self
     {
         Self::new(nonnull_debug_checked_from_ptr(ptr), dim, strides)
@@ -279,6 +283,7 @@ where D: Dimension
     /// but it's not a complete check.
     ///
     /// [`.offset()`]: https://doc.rust-lang.org/stable/std/primitive.pointer.html#method.offset
+    #[inline]
     pub unsafe fn from_shape_ptr<Sh>(shape: Sh, ptr: *mut A) -> Self
     where Sh: Into<StrideShape<D>>
     {
@@ -345,6 +350,7 @@ where D: Dimension
     ///
     /// **Panics** if `axis` or `index` is out of bounds.
     #[track_caller]
+    #[inline]
     pub fn split_at(self, axis: Axis, index: Ix) -> (Self, Self)
     {
         let (left, right) = self.into_raw_view().split_at(axis, index);

src/impl_views/constructors.rs

@@ -110,6 +110,7 @@ where D: Dimension
     /// but it's not a complete check.
     ///
     /// [`.offset()`]: https://doc.rust-lang.org/stable/std/primitive.pointer.html#method.offset
+    #[inline]
     pub unsafe fn from_shape_ptr<Sh>(shape: Sh, ptr: *const A) -> Self
     where Sh: Into<StrideShape<D>>
     {
@@ -212,6 +213,7 @@ where D: Dimension
     /// but it's not a complete check.
     ///
     /// [`.offset()`]: https://doc.rust-lang.org/stable/std/primitive.pointer.html#method.offset
+    #[inline]
     pub unsafe fn from_shape_ptr<Sh>(shape: Sh, ptr: *mut A) -> Self
     where Sh: Into<StrideShape<D>>
     {

src/impl_views/conversions.rs

@@ -62,6 +62,7 @@ where D: Dimension
     }
 
     /// Converts to a raw array view.
+    #[inline]
     pub(crate) fn into_raw_view(self) -> RawArrayView<A, D>
     {
         unsafe { RawArrayView::new(self.ptr, self.dim, self.strides) }

src/impl_views/splitting.rs

@@ -88,6 +88,7 @@ where D: Dimension
     ///                                      along Axis(1)
     /// ```
     #[track_caller]
+    #[inline]
     pub fn split_at(self, axis: Axis, index: Ix) -> (Self, Self)
     {
         unsafe {
@@ -137,6 +138,7 @@ where D: Dimension
     ///
     /// **Panics** if `axis` or `index` is out of bounds.
     #[track_caller]
+    #[inline]
     pub fn split_at(self, axis: Axis, index: Ix) -> (Self, Self)
     {
         unsafe {

src/shape_builder.rs

@@ -82,6 +82,7 @@ impl<D> Strides<D>
         }
     }
 
+    #[inline]
     pub(crate) fn is_custom(&self) -> bool
     {
         matches!(*self, Strides::Custom(_))