refactor: mark TskBox::into_raw safe (#623)

Author: molpopgen

src/sys/table_collection.rs

@@ -107,6 +107,6 @@ impl TableCollection {
     }
 
     pub fn into_raw(self) -> *mut tsk_table_collection_t {
-        unsafe { self.0.into_raw() }
+        self.0.into_raw()
     }
 }

src/sys/tskbox.rs

@@ -85,7 +85,7 @@ where
         }
     }
 
-    /// # Safety
+    /// # Note
     ///
     /// The returned pointer is no longer managed, meaing
     /// that it is the caller's responsibility to properly
@@ -94,7 +94,7 @@ where
     /// Failing to do so will may result in a
     /// memory leak.
     #[allow(dead_code)]
-    pub unsafe fn into_raw(self) -> *mut T {
+    pub fn into_raw(self) -> *mut T {
         let mut s = self;
         let rv = s.as_mut_ptr();
         s.owning = false;
@@ -189,7 +189,7 @@ fn test_into_raw_miri() {
     })
     .unwrap();
 
-    let p = unsafe { b.into_raw() };
+    let p = b.into_raw();
 
     unsafe { libc::free(p as *mut libc::c_void) }
 }