From 506105dc0dcfbbdec907f672e8d782b2f8604447 Mon Sep 17 00:00:00 2001 From: Zach Reizner Date: Tue, 10 Sep 2019 16:11:18 -0700 Subject: use `SharedMemory::{named, anon}` to replace `::new` The new constructors are shorter and omit the bare `None` in the `anon` call sites which gave no clues to the reader what the effect of that `None` was. This should improve readability. TEST=./build_test BUG=None Change-Id: I2e34e7df9a4ccc5da50edf4e963a6a42e3d84b22 Reviewed-on: https://chromium-review.googlesource.com/c/chromiumos/platform/crosvm/+/1797188 Reviewed-by: Daniel Verkamp Commit-Queue: Zach Reizner Tested-by: Zach Reizner Tested-by: kokoro --- gpu_display/src/gpu_display_wl.rs | 6 ++---- 1 file changed, 2 insertions(+), 4 deletions(-) (limited to 'gpu_display') diff --git a/gpu_display/src/gpu_display_wl.rs b/gpu_display/src/gpu_display_wl.rs index 670c429..3e1fe92 100644 --- a/gpu_display/src/gpu_display_wl.rs +++ b/gpu_display/src/gpu_display_wl.rs @@ -204,10 +204,8 @@ impl DisplayT for DisplayWl { let row_size = width * BYTES_PER_PIXEL; let fb_size = row_size * height; let buffer_size = round_up_to_page_size(fb_size as usize * BUFFER_COUNT); - let mut buffer_shm = SharedMemory::new(Some( - CStr::from_bytes_with_nul(b"GpuDisplaySurface\0").unwrap(), - )) - .map_err(GpuDisplayError::CreateShm)?; + let mut buffer_shm = + SharedMemory::named("GpuDisplaySurface").map_err(GpuDisplayError::CreateShm)?; buffer_shm .set_size(buffer_size as u64) .map_err(GpuDisplayError::SetSize)?; -- cgit 1.4.1