Deleted some constructors in h5pp::Hyperslab to avoid ambiguities

DavidAce · Nov 24, 2020 · e948625 · e948625
1 parent 628bd04
commit e948625
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/h5pp/include/h5pp/details/h5ppHyperslab.h b/h5pp/include/h5pp/details/h5ppHyperslab.h
@@ -23,6 +23,12 @@ namespace h5pp {
         Hyperslab(const DimsType &offset, const DimsType &extent, OptDimsType stride = std::nullopt, OptDimsType blocks = std::nullopt)
             : offset(offset), extent(extent), stride(std::move(stride)), blocks(std::move(blocks)) {}
 
+        // Delete some constructors to avoid ambiguity. Both offset and extent are required if any of them is to be given
+        Hyperslab(DimsType) = delete;
+        Hyperslab(OptDimsType) = delete;
+        Hyperslab(std::initializer_list<hsize_t>) = delete;
+        Hyperslab(std::vector<hsize_t>) = delete;
+
         explicit Hyperslab(const hid::h5s &space) {
             int rank = H5Sget_simple_extent_ndims(space);
             if(rank < 0) throw std::runtime_error("Could not read ndims on given space");