Rework SliceOrIndex to fit with valid selections

Author: mulimoen

src/hl/container.rs

@@ -69,15 +69,12 @@ impl<'a> Reader<'a> {
     where
         T: H5Type,
         S: TryInto<Selection>,
+        Error: From<S::Error>,
         D: ndarray::Dimension,
     {
         ensure!(!self.obj.is_attr(), "Slicing cannot be used on attribute datasets");
 
-        let selection = if let Ok(selection) = selection.try_into() {
-            selection
-        } else {
-            fail!("Selection is not valid for hdf5")
-        };
+        let selection = selection.try_into()?;
         let obj_space = self.obj.space()?;
 
         let out_shape = selection.out_shape(&obj_space.shape())?;
@@ -151,6 +148,7 @@ impl<'a> Reader<'a> {
     where
         T: H5Type,
         S: TryInto<Selection>,
+        Error: From<S::Error>,
     {
         self.read_slice(selection)
     }
@@ -168,6 +166,7 @@ impl<'a> Reader<'a> {
     where
         T: H5Type,
         S: TryInto<Selection>,
+        Error: From<S::Error>,
     {
         self.read_slice(selection)
     }
@@ -240,15 +239,12 @@ impl<'a> Writer<'a> {
         A: Into<ArrayView<'b, T, D>>,
         T: H5Type,
         S: TryInto<Selection>,
+        Error: From<S::Error>,
         D: ndarray::Dimension,
     {
         ensure!(!self.obj.is_attr(), "Slicing cannot be used on attribute datasets");
 
-        let selection = if let Ok(selection) = selection.try_into() {
-            selection
-        } else {
-            fail!("Selection is not valid for hdf5")
-        };
+        let selection = selection.try_into()?;
         let obj_space = self.obj.space()?;
 
         let out_shape = selection.out_shape(&obj_space.shape())?;
@@ -475,6 +471,7 @@ impl Container {
     where
         T: H5Type,
         S: TryInto<Selection>,
+        Error: From<S::Error>,
     {
         self.as_reader().read_slice_1d(selection)
     }
@@ -492,6 +489,7 @@ impl Container {
     where
         T: H5Type,
         S: TryInto<Selection>,
+        Error: From<S::Error>,
     {
         self.as_reader().read_slice_2d(selection)
     }
@@ -510,6 +508,7 @@ impl Container {
     where
         T: H5Type,
         S: TryInto<Selection>,
+        Error: From<S::Error>,
         D: ndarray::Dimension,
     {
         self.as_reader().read_slice(selection)
@@ -556,6 +555,7 @@ impl Container {
         A: Into<ArrayView<'b, T, D>>,
         T: H5Type,
         S: TryInto<Selection>,
+        Error: From<S::Error>,
         D: ndarray::Dimension,
     {
         self.as_writer().write_slice(arr, selection)