Commit 9f473538 authored by Benno Lossin's avatar Benno Lossin
Browse files

rust: pin-init: change `impl Zeroable for Option<NonNull<T>>` to `ZeroableOption for NonNull<T>` 

This brings it in line with references. It too is listed in [1].

Link: https://doc.rust-lang.org/stable/std/option/index.html#representation
Link: https://github.com/Rust-for-Linux/pin-init/pull/56/commits/8e52bf56ddc2190ce901d2f7c008ab8a64f653a9
Link: https://lore.kernel.org/all/20250523145125.523275-6-lossin@kernel.org


Signed-off-by: default avatarBenno Lossin <lossin@kernel.org>
parent e93a2386

rust/pin-init/src/lib.rs

+3 −1
Original line number Diff line number Diff line
@@ -1552,6 +1552,9 @@ unsafe impl<T> ZeroableOption for &T {} 
// SAFETY: `Option<&mut T>` is part of the option layout optimization guarantee:

// <https://doc.rust-lang.org/stable/std/option/index.html#representation>.

unsafe impl<T> ZeroableOption for &mut T {}

// SAFETY: `Option<NonNull<T>>` is part of the option layout optimization guarantee:

// <https://doc.rust-lang.org/stable/std/option/index.html#representation>.

unsafe impl<T> ZeroableOption for NonNull<T> {}



/// Create an initializer for a zeroed `T`.

///
@@ -1630,7 +1633,6 @@ macro_rules! impl_zeroable { 
    Option<NonZeroU128>, Option<NonZeroUsize>,

    Option<NonZeroI8>, Option<NonZeroI16>, Option<NonZeroI32>, Option<NonZeroI64>,

    Option<NonZeroI128>, Option<NonZeroIsize>,

    {<T>} Option<NonNull<T>>,



    // SAFETY: `null` pointer is valid.

    //