diff options
author | Daniel Verkamp <dverkamp@chromium.org> | 2019-10-15 13:13:40 -0700 |
---|---|---|
committer | Commit Bot <commit-bot@chromium.org> | 2019-10-17 03:35:35 +0000 |
commit | 130fbbe71db01378bf6aaeb104af66de86b4c3b8 (patch) | |
tree | e0871123e8828057519930ed98d02cc78cd42c73 | |
parent | 6494117e1766337f5d688b98bfc3df999932c3ac (diff) | |
download | crosvm-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.rs | 2 | ||||
-rw-r--r-- | devices/src/utils/event_loop.rs | 2 | ||||
-rw-r--r-- | devices/src/virtio/block.rs | 4 | ||||
-rw-r--r-- | gpu_display/src/lib.rs | 2 | ||||
-rw-r--r-- | qcow_utils/src/qcow_utils.rs | 2 | ||||
-rw-r--r-- | resources/src/gpu_allocator.rs | 3 | ||||
-rw-r--r-- | sys_util/src/lib.rs | 2 |
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, |