Fix HDF5 implicit copy (#920)

Delete copy constructors in HDF5 parsing classes.
Fixed DatasetIO move constructor.

fixes #916

Author: mmarineBlueQuartz

src/simplnx/Utilities/Parsing/HDF5/IO/AttributeIO.hpp

@@ -37,6 +37,9 @@ class SIMPLNX_EXPORT AttributeIO
    */
   AttributeIO(IdType objectId, const std::string& attrName);
 
+  AttributeIO(const AttributeIO& other) = delete;
+  AttributeIO(AttributeIO&& other) noexcept = default;
+
   /**
    * @brief Releases the wrapped HDF5 attribute.
    */
@@ -164,6 +167,9 @@ class SIMPLNX_EXPORT AttributeIO
   template <typename T>
   Result<> writeVector(const DimsVector& dims, const std::vector<T>& vector);
 
+  AttributeIO& operator=(const AttributeIO& other) = delete;
+  AttributeIO& operator=(AttributeIO&& other) noexcept = default;
+
 protected:
   /**
    * @brief Closes the HDF5 ID and resets it to 0.

src/simplnx/Utilities/Parsing/HDF5/IO/DatasetIO.cpp

@@ -24,7 +24,7 @@ DatasetIO::DatasetIO(IdType parentId, const std::string& datasetName)
 }
 
 DatasetIO::DatasetIO(DatasetIO&& other) noexcept
-: ObjectIO(other)
+: ObjectIO(std::move(other))
 , m_DatasetName(std::move(other.m_DatasetName))
 {
 }

src/simplnx/Utilities/Parsing/HDF5/IO/FileIO.hpp

@@ -61,6 +61,8 @@ class SIMPLNX_EXPORT FileIO : public GroupIO
    */
   FileIO(IdType fileId);
 
+  FileIO(const FileIO& rhs) = delete;
+
   /**
    * @brief Move constructor.
    * @param rhs
@@ -79,6 +81,9 @@ class SIMPLNX_EXPORT FileIO : public GroupIO
    */
   std::string getName() const override;
 
+  FileIO& operator=(const FileIO& rhs) = delete;
+  FileIO& operator=(FileIO&& rhs) noexcept = default;
+
 protected:
   /**
    * @brief Closes the HDF5 ID and resets it to 0.

src/simplnx/Utilities/Parsing/HDF5/IO/GroupIO.hpp

@@ -21,6 +21,9 @@ class SIMPLNX_EXPORT GroupIO : public ObjectIO
    */
   GroupIO(IdType parentId, const std::string& groupName);
 
+  GroupIO(const GroupIO& other) = delete;
+  GroupIO(GroupIO&& other) noexcept = default;
+
   /**
    * @brief Releases the wrapped HDF5 group.
    */
@@ -135,6 +138,9 @@ class SIMPLNX_EXPORT GroupIO : public ObjectIO
    */
   bool isDataset(const std::string& childName) const;
 
+  GroupIO& operator=(const GroupIO& other) = delete;
+  GroupIO& operator=(GroupIO&& other) noexcept = default;
+
 protected:
   /**
    * @brief Constructs a GroupWriter for use in derived classes. This

src/simplnx/Utilities/Parsing/HDF5/IO/ObjectIO.hpp

@@ -26,6 +26,9 @@ class SIMPLNX_EXPORT ObjectIO
    */
   ObjectIO(IdType parentId, const std::string& targetName);
 
+  ObjectIO(const ObjectIO& other) = delete;
+  ObjectIO(ObjectIO&& other) noexcept = default;
+
   /**
    * @brief Releases the wrapped HDF5 object.
    */
@@ -128,6 +131,9 @@ class SIMPLNX_EXPORT ObjectIO
    */
   AttributeIO createAttribute(const std::string& name);
 
+  ObjectIO& operator=(const ObjectIO& other) = delete;
+  ObjectIO& operator=(ObjectIO&& other) noexcept = default;
+
 protected:
   /**
    * @brief Constructs an ObjectIO for use in derived classes. This

src/simplnx/Utilities/Parsing/HDF5/Readers/AttributeReader.hpp

@@ -36,6 +36,9 @@ class SIMPLNX_EXPORT AttributeReader
    */
   AttributeReader(IdType objectId, const std::string& attrName);
 
+  AttributeReader(const AttributeReader& other) = delete;
+  AttributeReader(AttributeReader&& other) noexcept = default;
+
   /**
    * @brief Releases the wrapped HDF5 attribute.
    */
@@ -146,6 +149,9 @@ class SIMPLNX_EXPORT AttributeReader
    */
   std::string readAsString() const;
 
+  AttributeReader& operator=(const AttributeReader& other) = delete;
+  AttributeReader& operator=(AttributeReader&& other) noexcept = default;
+
 protected:
   /**
    * @brief Closes the HDF5 ID and resets it to 0.

src/simplnx/Utilities/Parsing/HDF5/Readers/DatasetReader.hpp

@@ -27,6 +27,9 @@ class SIMPLNX_EXPORT DatasetReader : public ObjectReader
    */
   DatasetReader(IdType parentId, const std::string& dataName);
 
+  DatasetReader(const DatasetReader& other) = delete;
+  DatasetReader(DatasetReader&& other) noexcept = default;
+
   /**
    * @brief Releases the HDF5 dataset.
    */
@@ -122,6 +125,9 @@ class SIMPLNX_EXPORT DatasetReader : public ObjectReader
 
   std::string getFilterName() const;
 
+  DatasetReader& operator=(const DatasetReader& other) = delete;
+  DatasetReader& operator=(DatasetReader&& other) noexcept = default;
+
 protected:
   /**
    * @brief Closes the HDF5 ID and resets it to 0.

src/simplnx/Utilities/Parsing/HDF5/Readers/FileReader.hpp

@@ -26,6 +26,9 @@ class SIMPLNX_EXPORT FileReader : public GroupReader
    */
   FileReader(IdType fileId);
 
+  FileReader(const FileReader& other) = delete;
+  FileReader(FileReader&& other) noexcept = default;
+
   /**
    * @brief Releases the HDF5 file ID.
    */
@@ -38,6 +41,9 @@ class SIMPLNX_EXPORT FileReader : public GroupReader
    */
   std::string getName() const override;
 
+  FileReader& operator=(const FileReader& other) = delete;
+  FileReader& operator=(FileReader&& other) noexcept = default;
+
 protected:
   /**
    * @brief Closes the HDF5 ID and resets it to 0.

src/simplnx/Utilities/Parsing/HDF5/Readers/GroupReader.hpp

@@ -19,6 +19,9 @@ class SIMPLNX_EXPORT GroupReader : public ObjectReader
    */
   GroupReader(IdType parentId, const std::string& groupName);
 
+  GroupReader(const GroupReader& other) = delete;
+  GroupReader(GroupReader&& other) noexcept = default;
+
   /**
    * @brief Releases the wrapped HDF5 group.
    */
@@ -87,6 +90,9 @@ class SIMPLNX_EXPORT GroupReader : public ObjectReader
    */
   bool isDataset(const std::string& childName) const;
 
+  GroupReader& operator=(const GroupReader& other) = delete;
+  GroupReader& operator=(GroupReader&& other) noexcept = default;
+
 protected:
   /**
    * @brief Closes the HDF5 ID and resets it to 0.

src/simplnx/Utilities/Parsing/HDF5/Readers/ObjectReader.hpp

@@ -24,6 +24,9 @@ class SIMPLNX_EXPORT ObjectReader
    */
   ObjectReader(IdType parentId, const std::string& targetName);
 
+  ObjectReader(const ObjectReader& other) = delete;
+  ObjectReader(ObjectReader&& other) noexcept = default;
+
   /**
    * @brief Releases the wrapped HDF5 object.
    */
@@ -96,6 +99,9 @@ class SIMPLNX_EXPORT ObjectReader
    */
   AttributeReader getAttributeByIdx(size_t idx) const;
 
+  ObjectReader& operator=(const ObjectReader& other) = delete;
+  ObjectReader& operator=(ObjectReader&& other) noexcept = default;
+
 protected:
   /**
    * @brief Constructs an ObjectReader for use in derived classes. This