Skip to content

Commit

Permalink
struct Rav1dPicture: Replace MaybeUninit::zeroed().assume_init()
Browse files Browse the repository at this point in the history
…with `Default::default()` as we already `impl Default` (#991)

Split off from #528.
  • Loading branch information
kkysen authored Apr 23, 2024
2 parents 8eb6932 + e60454a commit 0e3b16a
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions src/lib.rs
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,6 @@ use std::ffi::c_uint;
use std::ffi::c_ulong;
use std::ffi::c_void;
use std::mem;
use std::mem::MaybeUninit;
use std::process::abort;
use std::ptr;
use std::ptr::addr_of_mut;
Expand Down Expand Up @@ -583,7 +582,7 @@ pub unsafe extern "C" fn dav1d_get_picture(
validate_input!((!c.is_null(), EINVAL))?;
validate_input!((!out.is_null(), EINVAL))?;
let c = &mut *c;
let mut out_rust = MaybeUninit::zeroed().assume_init(); // TODO(kkysen) Temporary until we return it directly.
let mut out_rust = Default::default(); // TODO(kkysen) Temporary until we return it directly.
let result = rav1d_get_picture(c, &mut out_rust);
out.write(out_rust.into());
result
Expand Down Expand Up @@ -649,7 +648,7 @@ pub unsafe extern "C" fn dav1d_apply_grain(
// Don't `.update_rav1d()` [`Rav1dSequenceHeader`] because it's meant to be read-only.
// Don't `.update_rav1d()` [`Rav1dFrameHeader`] because it's meant to be read-only.
// Don't `.update_rav1d()` [`Rav1dITUTT35`] because we never read it.
let mut out_rust = MaybeUninit::zeroed().assume_init(); // TODO(kkysen) Temporary until we return it directly.
let mut out_rust = Default::default(); // TODO(kkysen) Temporary until we return it directly.
let in_rust = in_0.into();
let result = rav1d_apply_grain(c, &mut out_rust, &in_rust);
out.write(out_rust.into());
Expand Down

0 comments on commit 0e3b16a

Please sign in to comment.