mirrors/crosvm
1
0
Fork 0
mirror of https://chromium.googlesource.com/crosvm/crosvm synced 2025-02-05 18:20:34 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

devices: vios: remove unnecessary mut reference

Browse source 
The seq_socket_send() function only requires a non-mutable reference;
drop the mut.

Fixes clippy warning "the function `seq_socket_send` doesn't need a
mutable reference".

BUG=None
TEST=bin/clippy

Change-Id: I517a8d782601f33653db33c201666707beddd5ba
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/2885787
Tested-by: kokoro <noreply+kokoro@google.com>
Commit-Queue: Daniel Verkamp <dverkamp@chromium.org>
Reviewed-by: Zach Reizner <zachr@chromium.org>
Reviewed-by: Jorge Moreira Broche <jemoreira@google.com>
Reviewed-by: Dylan Reid <dgreid@chromium.org>
This commit is contained in:
Daniel Verkamp 2021-05-10 14:13:16 -07:00 committed by Commit Bot
parent c80319182d
commit b792ffdcec
2 changed files with 1 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

1
bin/clippy
View file

@ -41,7 +41,6 @@ SUPPRESS=(
range_plus_one range_plus_one
slow_vector_initialization slow_vector_initialization
unnecessary_filter_map unnecessary_filter_map
unnecessary_mut_passed
unneeded_field_pattern unneeded_field_pattern
useless_format useless_format
wrong_self_convention wrong_self_convention

2
devices/src/virtio/snd/vios_backend/shm_vios.rs
View file

@ -364,7 +364,7 @@ impl VioSClient {
fn send_cmd<T: DataInit>(&self, data: T) -> Result<()> { fn send_cmd<T: DataInit>(&self, data: T) -> Result<()> {
let mut control_socket_lock = self.control_socket.lock(); let mut control_socket_lock = self.control_socket.lock();
seq_socket_send(&mut *control_socket_lock, data)?; seq_socket_send(&*control_socket_lock, data)?;
recv_cmd_status(&mut *control_socket_lock) recv_cmd_status(&mut *control_socket_lock)
} }