diff options
author | Daniel Verkamp <dverkamp@chromium.org> | 2019-09-24 11:06:58 -0700 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2019-10-17 00:17:05 +0000 |
commit | 04a82c7be173b2068c4254ed4a129e24e9e3a2e4 (patch) | |
tree | 253fdffb4299b8ed379cec420c54b81c8b0d5865 /devices/src/pci/vfio_pci.rs | |
parent | a7b75c8a219eb98e49d995ea58e33e1832533e1b (diff) | |
download | crosvm-04a82c7be173b2068c4254ed4a129e24e9e3a2e4.tar crosvm-04a82c7be173b2068c4254ed4a129e24e9e3a2e4.tar.gz crosvm-04a82c7be173b2068c4254ed4a129e24e9e3a2e4.tar.bz2 crosvm-04a82c7be173b2068c4254ed4a129e24e9e3a2e4.tar.lz crosvm-04a82c7be173b2068c4254ed4a129e24e9e3a2e4.tar.xz crosvm-04a82c7be173b2068c4254ed4a129e24e9e3a2e4.tar.zst crosvm-04a82c7be173b2068c4254ed4a129e24e9e3a2e4.zip |
devices: vfio: fix clippy warnings
Fix boxed_local, const_static_lifetime, useless_format, and redundant_closure clippy warnings in the VFIO code. This fixes all clippy warnings except a single instance of let_and_return in VfioPciDevice::keep_fds(), since that code is modified in an upcoming patch. BUG=None TEST=./build_test.py TEST=bin/clippy Change-Id: I548adbc6b92448fc0db82ed72214d73b0eabaf5c Signed-off-by: Daniel Verkamp <dverkamp@chromium.org> Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/1822697 Reviewed-by: Xiong Zhang <xiong.y.zhang@intel.corp-partner.google.com> Reviewed-by: Zach Reizner <zachr@chromium.org> Tested-by: Xiong Zhang <xiong.y.zhang@intel.corp-partner.google.com> Tested-by: kokoro <noreply+kokoro@google.com>
Diffstat (limited to 'devices/src/pci/vfio_pci.rs')
-rw-r--r-- | devices/src/pci/vfio_pci.rs | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/devices/src/pci/vfio_pci.rs b/devices/src/pci/vfio_pci.rs index 9073312..5b0474a 100644 --- a/devices/src/pci/vfio_pci.rs +++ b/devices/src/pci/vfio_pci.rs @@ -100,8 +100,8 @@ pub struct VfioPciDevice { impl VfioPciDevice { /// Constructs a new Vfio Pci device for the give Vfio device - pub fn new(device: Box<VfioDevice>) -> Self { - let dev = Arc::new(*device); + pub fn new(device: VfioDevice) -> Self { + let dev = Arc::new(device); let config = VfioPciConfig::new(Arc::clone(&dev)); VfioPciDevice { device: dev, @@ -131,7 +131,7 @@ impl VfioPciDevice { impl PciDevice for VfioPciDevice { fn debug_label(&self) -> String { - format!("vfio pci device") + "vfio pci device".to_string() } fn assign_bus_dev(&mut self, bus: u8, device: u8) { |