Rename zero_index_with_ndim -> zeros and make it public

Author: jturner314

src/dimension/dimension_trait.rs

@@ -150,20 +150,19 @@ pub trait Dimension : Clone + Eq + Debug + Send + Sync + Default +
         strides
     }
 
-    #[doc(hidden)]
-    // Return an index of same dimensionality
-    fn zero_index(&self) -> Self {
-        Self::default()
-    }
-
-    #[doc(hidden)]
-    /// Return an index of same type and with the specified dimensionality.
+    /// Creates a dimension of all zeros with the specified ndim.
     ///
     /// This method is useful for generalizing over fixed-size and
     /// variable-size dimension representations.
     ///
     /// **Panics** if `Self` has a fixed size that is not `ndim`.
-    fn zero_index_with_ndim(ndim: usize) -> Self;
+    fn zeros(ndim: usize) -> Self;
+
+    #[doc(hidden)]
+    // Return an index of same dimensionality
+    fn zero_index(&self) -> Self {
+        Self::default()
+    }
 
     #[doc(hidden)]
     #[inline]
@@ -380,7 +379,7 @@ impl Dimension for Dim<[Ix; 0]> {
     #[inline]
     fn into_pattern(self) -> Self::Pattern { }
     #[inline]
-    fn zero_index_with_ndim(ndim: usize) -> Self {
+    fn zeros(ndim: usize) -> Self {
         assert_eq!(ndim, 0);
         Self::default()
     }
@@ -416,7 +415,7 @@ impl Dimension for Dim<[Ix; 1]> {
         get!(&self, 0)
     }
     #[inline]
-    fn zero_index_with_ndim(ndim: usize) -> Self {
+    fn zeros(ndim: usize) -> Self {
         assert_eq!(ndim, 1);
         Self::default()
     }
@@ -510,7 +509,7 @@ impl Dimension for Dim<[Ix; 2]> {
     #[inline]
     fn slice_mut(&mut self) -> &mut [Ix] { self.ixm() }
     #[inline]
-    fn zero_index_with_ndim(ndim: usize) -> Self {
+    fn zeros(ndim: usize) -> Self {
         assert_eq!(ndim, 2);
         Self::default()
     }
@@ -655,7 +654,7 @@ impl Dimension for Dim<[Ix; 3]> {
     }
 
     #[inline]
-    fn zero_index_with_ndim(ndim: usize) -> Self {
+    fn zeros(ndim: usize) -> Self {
         assert_eq!(ndim, 3);
         Self::default()
     }
@@ -764,7 +763,7 @@ macro_rules! large_dim {
             #[inline]
             fn slice_mut(&mut self) -> &mut [Ix] { self.ixm() }
             #[inline]
-            fn zero_index_with_ndim(ndim: usize) -> Self {
+            fn zeros(ndim: usize) -> Self {
                 assert_eq!(ndim, $n);
                 Self::default()
             }
@@ -822,7 +821,7 @@ impl Dimension for IxDyn
     }
 
     #[inline]
-    fn zero_index_with_ndim(ndim: usize) -> Self {
+    fn zeros(ndim: usize) -> Self {
         IxDyn::zeros(ndim)
     }
 

src/impl_methods.rs

@@ -295,8 +295,8 @@ impl<A, S, D> ArrayBase<S, D> where S: Data<Elem=A>, D: Dimension
 
         // Copy the dim and strides that remain after removing the subview axes.
         let out_ndim = info.out_ndim();
-        let mut new_dim = Do::zero_index_with_ndim(out_ndim);
-        let mut new_strides = Do::zero_index_with_ndim(out_ndim);
+        let mut new_dim = Do::zeros(out_ndim);
+        let mut new_strides = Do::zeros(out_ndim);
         izip!(self.dim.slice(), self.strides.slice(), indices)
             .filter_map(|(d, s, slice_or_index)| match slice_or_index {
                 &SliceOrIndex::Slice {..} => Some((d, s)),
@@ -1378,7 +1378,7 @@ impl<A, S, D> ArrayBase<S, D> where S: Data<Elem=A>, D: Dimension
     {
         let axes = axes.into_dimension();
         // Ensure that each axis is used exactly once.
-        let mut usage_counts = D::zero_index_with_ndim(self.ndim());
+        let mut usage_counts = D::zeros(self.ndim());
         for axis in axes.slice() {
             usage_counts[*axis] += 1;
         }
@@ -1387,7 +1387,7 @@ impl<A, S, D> ArrayBase<S, D> where S: Data<Elem=A>, D: Dimension
         }
         // Determine the new shape and strides.
         let mut new_dim = usage_counts; // reuse to avoid an allocation
-        let mut new_strides = D::zero_index_with_ndim(self.ndim());
+        let mut new_strides = D::zeros(self.ndim());
         {
             let dim = self.dim.slice();
             let strides = self.strides.slice();