summary refs log tree commit diff
diff options
context:
space:
mode:
authorDaniel Verkamp <dverkamp@chromium.org>2019-10-15 13:13:40 -0700
committerCommit Bot <commit-bot@chromium.org>2019-10-17 03:35:35 +0000
commit130fbbe71db01378bf6aaeb104af66de86b4c3b8 (patch)
treee0871123e8828057519930ed98d02cc78cd42c73
parent6494117e1766337f5d688b98bfc3df999932c3ac (diff)
downloadcrosvm-130fbbe71db01378bf6aaeb104af66de86b4c3b8.tar
crosvm-130fbbe71db01378bf6aaeb104af66de86b4c3b8.tar.gz
crosvm-130fbbe71db01378bf6aaeb104af66de86b4c3b8.tar.bz2
crosvm-130fbbe71db01378bf6aaeb104af66de86b4c3b8.tar.lz
crosvm-130fbbe71db01378bf6aaeb104af66de86b4c3b8.tar.xz
crosvm-130fbbe71db01378bf6aaeb104af66de86b4c3b8.tar.zst
crosvm-130fbbe71db01378bf6aaeb104af66de86b4c3b8.zip
Add explicit `dyn` for trait objects
Fix "trait objects without an explicit `dyn` are deprecated" warnings
introduced in Rust 1.38.

BUG=None
TEST=emerge-nami crosvm

Change-Id: I8ca6aa747475268ae898adddd5d091d401326ceb
Signed-off-by: Daniel Verkamp <dverkamp@chromium.org>
Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/1862999
Reviewed-by: Dylan Reid <dgreid@chromium.org>
Reviewed-by: Stephen Barber <smbarber@chromium.org>
Tested-by: kokoro <noreply+kokoro@google.com>
-rw-r--r--devices/src/register_space/register.rs2
-rw-r--r--devices/src/utils/event_loop.rs2
-rw-r--r--devices/src/virtio/block.rs4
-rw-r--r--gpu_display/src/lib.rs2
-rw-r--r--qcow_utils/src/qcow_utils.rs2
-rw-r--r--resources/src/gpu_allocator.rs3
-rw-r--r--sys_util/src/lib.rs2
7 files changed, 9 insertions, 8 deletions
diff --git a/devices/src/register_space/register.rs b/devices/src/register_space/register.rs
index 268b945..c2f5184 100644
--- a/devices/src/register_space/register.rs
+++ b/devices/src/register_space/register.rs
@@ -448,7 +448,7 @@ mod tests {
 
     #[test]
     fn static_register_interface_test() {
-        let r: Box<RegisterInterface> = Box::new(static_register! {
+        let r: Box<dyn RegisterInterface> = Box::new(static_register! {
             ty: u8,
             offset: 3,
             value: 32,
diff --git a/devices/src/utils/event_loop.rs b/devices/src/utils/event_loop.rs
index 405c543..6db8d0e 100644
--- a/devices/src/utils/event_loop.rs
+++ b/devices/src/utils/event_loop.rs
@@ -236,7 +236,7 @@ mod tests {
             cvar: Condvar::new(),
             evt,
         });
-        let t: Arc<EventHandler> = h.clone();
+        let t: Arc<dyn EventHandler> = h.clone();
         l.add_event(
             &h.evt,
             WatchingEvents::empty().set_read(),
diff --git a/devices/src/virtio/block.rs b/devices/src/virtio/block.rs
index b171864..a76a3b3 100644
--- a/devices/src/virtio/block.rs
+++ b/devices/src/virtio/block.rs
@@ -284,7 +284,7 @@ impl Worker {
     fn process_one_request(
         avail_desc: DescriptorChain,
         read_only: bool,
-        disk: &mut DiskFile,
+        disk: &mut dyn DiskFile,
         disk_size: u64,
         flush_timer: &mut TimerFd,
         flush_timer_armed: &mut bool,
@@ -571,7 +571,7 @@ impl Block {
     fn execute_request(
         avail_desc: DescriptorChain,
         read_only: bool,
-        disk: &mut DiskFile,
+        disk: &mut dyn DiskFile,
         disk_size: u64,
         flush_timer: &mut TimerFd,
         flush_timer_armed: &mut bool,
diff --git a/gpu_display/src/lib.rs b/gpu_display/src/lib.rs
index 1bf9578..3067c0e 100644
--- a/gpu_display/src/lib.rs
+++ b/gpu_display/src/lib.rs
@@ -159,7 +159,7 @@ trait DisplayT: AsRawFd {
 /// The user of `GpuDisplay` can use `AsRawFd` to poll on the compositor connection's file
 /// descriptor. When the connection is readable, `dispatch_events` can be called to process it.
 pub struct GpuDisplay {
-    inner: Box<DisplayT>,
+    inner: Box<dyn DisplayT>,
 }
 
 impl GpuDisplay {
diff --git a/qcow_utils/src/qcow_utils.rs b/qcow_utils/src/qcow_utils.rs
index 1dcf2a4..6f11c66 100644
--- a/qcow_utils/src/qcow_utils.rs
+++ b/qcow_utils/src/qcow_utils.rs
@@ -77,7 +77,7 @@ pub unsafe extern "C" fn expand_disk_image(path: *const c_char, virtual_size: u6
         Err(_) => return -EINVAL,
     };
 
-    let mut disk_image: Box<DiskFile> = match image_type {
+    let mut disk_image: Box<dyn DiskFile> = match image_type {
         ImageType::Raw => Box::new(raw_image),
         ImageType::Qcow2 => match QcowFile::from(raw_image) {
             Ok(f) => Box::new(f),
diff --git a/resources/src/gpu_allocator.rs b/resources/src/gpu_allocator.rs
index a44d3c8..478d0d4 100644
--- a/resources/src/gpu_allocator.rs
+++ b/resources/src/gpu_allocator.rs
@@ -111,7 +111,8 @@ impl GpuMemoryAllocator for GpuBufferDevice {
 }
 
 #[cfg(feature = "wl-dmabuf")]
-pub fn create_gpu_memory_allocator() -> Result<Option<Box<GpuMemoryAllocator>>, GpuAllocatorError> {
+pub fn create_gpu_memory_allocator(
+) -> Result<Option<Box<dyn GpuMemoryAllocator>>, GpuAllocatorError> {
     let undesired: &[&str] = &["vgem", "pvr"];
     let fd = gpu_buffer::rendernode::open_device(undesired)
         .map_err(|_| GpuAllocatorError::OpenGpuBufferDevice)?;
diff --git a/sys_util/src/lib.rs b/sys_util/src/lib.rs
index 0dab89d..8428429 100644
--- a/sys_util/src/lib.rs
+++ b/sys_util/src/lib.rs
@@ -326,7 +326,7 @@ pub fn validate_raw_fd(raw_fd: RawFd) -> Result<RawFd> {
 ///
 /// On an error, such as an invalid or incompatible FD, this will return false, which can not be
 /// distinguished from a non-ready to read FD.
-pub fn poll_in(fd: &AsRawFd) -> bool {
+pub fn poll_in(fd: &dyn AsRawFd) -> bool {
     let mut fds = libc::pollfd {
         fd: fd.as_raw_fd(),
         events: libc::POLLIN,