From 8fd1ec600d0f4ad1ba0e13cfe446edb82877dad7 Mon Sep 17 00:00:00 2001 From: Tatsuyuki Ishi Date: Wed, 3 Aug 2022 12:49:46 +0900 Subject: [PATCH] media: ffmpeg: Replace #[allow(dead_code)] with underscore prefix. It's slightly less verbose and more idiomatic to do it this way. BUG=None TEST=cargo build Change-Id: I1f67f88543b5e65cb187406179e463b9182fade5 Reviewed-on: https://chromium-review.googlesource.com/c/crosvm/crosvm/+/3809196 Commit-Queue: Tatsuyuki Ishi Tested-by: Tatsuyuki Ishi Reviewed-by: Junichi Uekawa Reviewed-by: Alexandre Courbot --- media/ffmpeg/src/avcodec.rs | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) diff --git a/media/ffmpeg/src/avcodec.rs b/media/ffmpeg/src/avcodec.rs index 2f23e53ab1..4bc51ba49e 100644 --- a/media/ffmpeg/src/avcodec.rs +++ b/media/ffmpeg/src/avcodec.rs @@ -367,8 +367,7 @@ pub struct AvPacket<'a> { packet: ffi::AVPacket, /// This is just used to drop the `AvBuffer` if needed upon destruction and is not referenced /// otherwise. - #[allow(dead_code)] - ownership: AvPacketOwnership<'a>, + _ownership: AvPacketOwnership<'a>, } #[derive(Debug, ThisError)] @@ -395,7 +394,7 @@ impl<'a> AvPacket<'a> { // Safe because all the other elements of this struct can be zeroed. ..unsafe { std::mem::zeroed() } }, - ownership: AvPacketOwnership::Borrowed(PhantomData), + _ownership: AvPacketOwnership::Borrowed(PhantomData), } } @@ -422,7 +421,7 @@ impl<'a> AvPacket<'a> { // Safe because all the other elements of this struct can be zeroed. ..unsafe { std::mem::zeroed() } }, - ownership: AvPacketOwnership::Owned(av_buffer), + _ownership: AvPacketOwnership::Owned(av_buffer), }; Ok(ret)