From 54dce6fb9c8d97010baa29b38c4514e2112a3ce4 Mon Sep 17 00:00:00 2001 From: Alexandre Courbot Date: Tue, 15 Sep 2020 20:42:19 +0900 Subject: [PATCH] devices: virtio: video: remove redundant clone We are taking ownership of params here, so no need to clone. BUG=None TEST=Video plays in Youtube. Change-Id: I1013e669cde7e5d0001992d7620643054c03d87d Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/2409489 Tested-by: Alexandre Courbot Tested-by: kokoro Auto-Submit: Alexandre Courbot Reviewed-by: Keiichi Watanabe Commit-Queue: Alexandre Courbot --- devices/src/virtio/video/decoder/mod.rs | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/devices/src/virtio/video/decoder/mod.rs b/devices/src/virtio/video/decoder/mod.rs index ee1b6b9c9f..5cb40753aa 100644 --- a/devices/src/virtio/video/decoder/mod.rs +++ b/devices/src/virtio/video/decoder/mod.rs @@ -604,7 +604,7 @@ impl<'a> Decoder<'a> { // Only a few parameters can be changed by the guest. ctx.in_params.format = params.format; - ctx.in_params.plane_formats = params.plane_formats.clone(); + ctx.in_params.plane_formats = params.plane_formats; } QueueType::Output => { // The guest cannot update parameters for output queue in the decoder.