refactor: Move References to Module in H5Types

Author: JamesWiresmith

hdf5-types/src/h5type.rs

@@ -4,6 +4,7 @@ use std::os::raw::c_void;
 use std::ptr;
 
 use crate::array::VarLenArray;
+use crate::references::Reference;
 use crate::string::{FixedAscii, FixedUnicode, VarLenAscii, VarLenUnicode};
 
 #[allow(non_camel_case_types)]
@@ -381,25 +382,6 @@ unsafe impl H5Type for VarLenUnicode {
     }
 }
 
-#[derive(Copy, Clone, Debug, PartialEq, Eq)]
-pub enum Reference {
-    Object,
-    Region,
-    #[cfg(feature = "1.12.0")]
-    Std,
-}
-
-impl Reference {
-    fn size(self) -> usize {
-        match self {
-            Self::Object => mem::size_of::<hdf5_sys::h5r::hobj_ref_t>(),
-            Self::Region => mem::size_of::<hdf5_sys::h5r::hdset_reg_ref_t>(),
-            #[cfg(feature = "1.12.0")]
-            Self::Std => mem::size_of::<hdf5_sys::h5r::H5R_ref_t>(),
-        }
-    }
-}
-
 #[cfg(test)]
 pub mod tests {
     use super::TypeDescriptor as TD;

hdf5-types/src/lib.rs

@@ -29,9 +29,9 @@ mod complex;
 pub use self::array::VarLenArray;
 pub use self::dyn_value::{DynValue, OwnedDynValue};
 pub use self::h5type::{
-    CompoundField, CompoundType, EnumMember, EnumType, FloatSize, H5Type, IntSize, Reference,
-    TypeDescriptor,
+    CompoundField, CompoundType, EnumMember, EnumType, FloatSize, H5Type, IntSize, TypeDescriptor,
 };
+pub use self::references::Reference;
 pub use self::string::{FixedAscii, FixedUnicode, StringError, VarLenAscii, VarLenUnicode};
 
 pub(crate) unsafe fn malloc(n: usize) -> *mut core::ffi::c_void {

hdf5-types/src/references.rs

@@ -1 +1,22 @@
-// TODO: move references to this location
+//! Types for references.
+
+use std::mem;
+
+#[derive(Copy, Clone, Debug, PartialEq, Eq)]
+pub enum Reference {
+    Object,
+    Region,
+    #[cfg(feature = "1.12.0")]
+    Std,
+}
+
+impl Reference {
+    pub fn size(self) -> usize {
+        match self {
+            Self::Object => mem::size_of::<hdf5_sys::h5r::hobj_ref_t>(),
+            Self::Region => mem::size_of::<hdf5_sys::h5r::hdset_reg_ref_t>(),
+            #[cfg(feature = "1.12.0")]
+            Self::Std => mem::size_of::<hdf5_sys::h5r::H5R_ref_t>(),
+        }
+    }
+}