fn rav1d_data_ref: Cleanup (#643)

memorysafety · Dec 20, 2023 · 5680365 · 5680365
2 parents 3ae9c3b + 38a8c11
commit 5680365
Showing 1 changed file with 7 additions and 13 deletions.
diff --git a/src/data.rs b/src/data.rs
@@ -69,26 +69,20 @@ pub(crate) unsafe fn rav1d_data_wrap_user_data_internal(
     Ok(())
 }
 
-pub(crate) unsafe fn rav1d_data_ref(dst: *mut Rav1dData, src: *const Rav1dData) {
-    if validate_input!(!dst.is_null()).is_err() {
-        return;
-    }
+pub(crate) unsafe fn rav1d_data_ref(dst: &mut Rav1dData, src: &Rav1dData) {
     if validate_input!((*dst).data.is_null()).is_err() {
         return;
     }
-    if validate_input!(!src.is_null()).is_err() {
-        return;
-    }
-    if !((*src).r#ref).is_null() {
-        if validate_input!(!(*src).data.is_null()).is_err() {
+    if !src.r#ref.is_null() {
+        if validate_input!(!src.data.is_null()).is_err() {
             return;
         }
-        rav1d_ref_inc((*src).r#ref);
+        rav1d_ref_inc(src.r#ref);
     }
-    if !((*src).m.user_data.r#ref).is_null() {
-        rav1d_ref_inc((*src).m.user_data.r#ref);
+    if !src.m.user_data.r#ref.is_null() {
+        rav1d_ref_inc(src.m.user_data.r#ref);
     }
-    *dst = (*src).clone();
+    *dst = src.clone();
 }
 
 pub(crate) unsafe fn rav1d_data_props_copy(dst: *mut Rav1dDataProps, src: *const Rav1dDataProps) {