PropertyList.clone() -> PropertyList.copy()

Author: aldanor

hdf5-rs/src/hl/plist.rs

@@ -46,13 +46,6 @@ impl Deref for PropertyList {
     }
 }
 
-impl Clone for PropertyList {
-    fn clone(&self) -> Self {
-        let id = h5call!(H5Pcopy(self.id())).unwrap_or(H5I_INVALID_HID);
-        Self::from_id(id).ok().unwrap_or_else(Self::invalid)
-    }
-}
-
 impl PartialEq for PropertyList {
     fn eq(&self, other: &Self) -> bool {
         h5call!(H5Pequal(self.id(), other.id())).unwrap_or(0) == 1
@@ -156,6 +149,11 @@ impl FromStr for PropertyListClass {
 }
 
 impl PropertyList {
+    /// Copies the property list.
+    pub fn copy(&self) -> Result<Self> {
+        Self::from_id(h5call!(H5Pcopy(self.id()))?)
+    }
+
     /// Queries whether a property name exists in the property list.
     pub fn has(&self, property: &str) -> bool {
         to_cstring(property)
@@ -244,7 +242,7 @@ pub mod tests {
     pub fn test_clone() {
         let (fapl, _) = make_plists();
         assert!(fapl.is_valid());
-        let fapl_c = fapl.clone();
+        let fapl_c = fapl.copy().unwrap();
         assert!(fapl.is_valid());
         assert!(fapl_c.is_valid());
         assert_eq!(fapl.refcount(), 1);