diff --git a/examples/multithreaded.rs b/examples/multithreaded.rs index 388a4af5dd..c91ce5a593 100644 --- a/examples/multithreaded.rs +++ b/examples/multithreaded.rs @@ -12,7 +12,7 @@ fn main() { }; const WINDOW_COUNT: usize = 3; - const WINDOW_SIZE: PhysicalSize = PhysicalSize::new(600, 400); + const WINDOW_SIZE: PhysicalSize = PhysicalSize::new(600, 400); env_logger::init(); let event_loop = EventLoop::new(); @@ -108,9 +108,9 @@ fn main() { M => window.set_maximized(state), P => window.set_outer_position({ let mut position = window.outer_position().unwrap(); - let sign = if state { 1.0 } else { -1.0 }; - position.x += 10.0 * sign; - position.y += 10.0 * sign; + let sign = if state { 1 } else { -1 }; + position.x += 10 * sign; + position.y += 10 * sign; position }), Q => window.request_redraw(), @@ -126,10 +126,7 @@ fn main() { if let Size::Physical(size) = WINDOW_SIZE.into() { window .set_cursor_position(Position::Physical( - PhysicalPosition::new( - size.width as f64 / 2.0, - size.height as f64 / 2.0, - ), + PhysicalPosition::new(size.width / 2, size.height / 2), )) .unwrap() } diff --git a/src/dpi.rs b/src/dpi.rs index 4fc89e528d..a1a0491cbd 100644 --- a/src/dpi.rs +++ b/src/dpi.rs @@ -76,6 +76,54 @@ //! If you never received any [`HiDpiFactorChanged`](crate::event::WindowEvent::HiDpiFactorChanged) events, //! then your window's DPI factor is 1. +pub trait Pixel: Copy + Into { + fn from_f64(f: f64) -> Self; + fn cast(self) -> P { + P::from_f64(self.into()) + } +} + +impl Pixel for u8 { + fn from_f64(f: f64) -> Self { + f.round() as u8 + } +} +impl Pixel for u16 { + fn from_f64(f: f64) -> Self { + f.round() as u16 + } +} +impl Pixel for u32 { + fn from_f64(f: f64) -> Self { + f.round() as u32 + } +} +impl Pixel for i8 { + fn from_f64(f: f64) -> Self { + f.round() as i8 + } +} +impl Pixel for i16 { + fn from_f64(f: f64) -> Self { + f.round() as i16 + } +} +impl Pixel for i32 { + fn from_f64(f: f64) -> Self { + f.round() as i32 + } +} +impl Pixel for f32 { + fn from_f64(f: f64) -> Self { + f as f32 + } +} +impl Pixel for f64 { + fn from_f64(f: f64) -> Self { + f + } +} + /// Checks that the DPI factor is a normal positive `f64`. /// /// All functions that take a DPI factor assert that this will return `true`. If you're sourcing DPI factors from @@ -93,57 +141,53 @@ pub fn validate_hidpi_factor(dpi_factor: f64) -> bool { /// does the rounding for you. #[derive(Debug, Copy, Clone, PartialEq)] #[cfg_attr(feature = "serde", derive(Serialize, Deserialize))] -pub struct LogicalPosition { - pub x: f64, - pub y: f64, +pub struct LogicalPosition

{ + pub x: P, + pub y: P, } -impl LogicalPosition { +impl

LogicalPosition

{ #[inline] - pub const fn new(x: f64, y: f64) -> Self { + pub const fn new(x: P, y: P) -> Self { LogicalPosition { x, y } } +} +impl LogicalPosition

{ #[inline] - pub fn from_physical>(physical: T, dpi_factor: f64) -> Self { + pub fn from_physical>, X: Pixel>( + physical: T, + dpi_factor: f64, + ) -> Self { physical.into().to_logical(dpi_factor) } #[inline] - pub fn to_physical(&self, dpi_factor: f64) -> PhysicalPosition { + pub fn to_physical(&self, dpi_factor: f64) -> PhysicalPosition { assert!(validate_hidpi_factor(dpi_factor)); - let x = self.x * dpi_factor; - let y = self.y * dpi_factor; - PhysicalPosition::new(x, y) + let x = self.x.into() * dpi_factor; + let y = self.y.into() * dpi_factor; + PhysicalPosition::new(x, y).cast() } -} - -impl From<(f64, f64)> for LogicalPosition { - #[inline] - fn from((x, y): (f64, f64)) -> Self { - Self::new(x, y) - } -} -impl From<(i32, i32)> for LogicalPosition { #[inline] - fn from((x, y): (i32, i32)) -> Self { - Self::new(x as f64, y as f64) + pub fn cast(&self) -> LogicalPosition { + LogicalPosition { + x: self.x.cast(), + y: self.y.cast(), + } } } -impl Into<(f64, f64)> for LogicalPosition { - #[inline] - fn into(self) -> (f64, f64) { - (self.x, self.y) +impl From<(X, X)> for LogicalPosition

{ + fn from((x, y): (X, X)) -> LogicalPosition

{ + LogicalPosition::new(x.cast(), y.cast()) } } -impl Into<(i32, i32)> for LogicalPosition { - /// Note that this rounds instead of truncating. - #[inline] - fn into(self) -> (i32, i32) { - (self.x.round() as _, self.y.round() as _) +impl Into<(X, X)> for LogicalPosition

{ + fn into(self: Self) -> (X, X) { + (self.x.cast(), self.y.cast()) } } @@ -154,57 +198,53 @@ impl Into<(i32, i32)> for LogicalPosition { /// does the rounding for you. #[derive(Debug, Copy, Clone, PartialEq)] #[cfg_attr(feature = "serde", derive(Serialize, Deserialize))] -pub struct PhysicalPosition { - pub x: f64, - pub y: f64, +pub struct PhysicalPosition

{ + pub x: P, + pub y: P, } -impl PhysicalPosition { +impl

PhysicalPosition

{ #[inline] - pub const fn new(x: f64, y: f64) -> Self { + pub const fn new(x: P, y: P) -> Self { PhysicalPosition { x, y } } +} +impl PhysicalPosition

{ #[inline] - pub fn from_logical>(logical: T, dpi_factor: f64) -> Self { + pub fn from_logical>, X: Pixel>( + logical: T, + dpi_factor: f64, + ) -> Self { logical.into().to_physical(dpi_factor) } #[inline] - pub fn to_logical(&self, dpi_factor: f64) -> LogicalPosition { + pub fn to_logical(&self, dpi_factor: f64) -> LogicalPosition { assert!(validate_hidpi_factor(dpi_factor)); - let x = self.x / dpi_factor; - let y = self.y / dpi_factor; - LogicalPosition::new(x, y) - } -} - -impl From<(f64, f64)> for PhysicalPosition { - #[inline] - fn from((x, y): (f64, f64)) -> Self { - Self::new(x, y) + let x = self.x.into() / dpi_factor; + let y = self.y.into() / dpi_factor; + LogicalPosition::new(x, y).cast() } -} -impl From<(i32, i32)> for PhysicalPosition { #[inline] - fn from((x, y): (i32, i32)) -> Self { - Self::new(x as f64, y as f64) + pub fn cast(&self) -> PhysicalPosition { + PhysicalPosition { + x: self.x.cast(), + y: self.y.cast(), + } } } -impl Into<(f64, f64)> for PhysicalPosition { - #[inline] - fn into(self) -> (f64, f64) { - (self.x, self.y) +impl From<(X, X)> for PhysicalPosition

{ + fn from((x, y): (X, X)) -> PhysicalPosition

{ + PhysicalPosition::new(x.cast(), y.cast()) } } -impl Into<(i32, i32)> for PhysicalPosition { - /// Note that this rounds instead of truncating. - #[inline] - fn into(self) -> (i32, i32) { - (self.x.round() as _, self.y.round() as _) +impl Into<(X, X)> for PhysicalPosition

{ + fn into(self: Self) -> (X, X) { + (self.x.cast(), self.y.cast()) } } @@ -215,108 +255,108 @@ impl Into<(i32, i32)> for PhysicalPosition { /// does the rounding for you. #[derive(Debug, Copy, Clone, PartialEq)] #[cfg_attr(feature = "serde", derive(Serialize, Deserialize))] -pub struct LogicalSize { - pub width: f64, - pub height: f64, +pub struct LogicalSize

{ + pub width: P, + pub height: P, } -impl LogicalSize { +impl

LogicalSize

{ #[inline] - pub const fn new(width: f64, height: f64) -> Self { + pub const fn new(width: P, height: P) -> Self { LogicalSize { width, height } } +} +impl LogicalSize

{ #[inline] - pub fn from_physical>(physical: T, dpi_factor: f64) -> Self { + pub fn from_physical>, X: Pixel>(physical: T, dpi_factor: f64) -> Self { physical.into().to_logical(dpi_factor) } #[inline] - pub fn to_physical(&self, dpi_factor: f64) -> PhysicalSize { + pub fn to_physical(&self, dpi_factor: f64) -> PhysicalSize { assert!(validate_hidpi_factor(dpi_factor)); - let width = self.width * dpi_factor; - let height = self.height * dpi_factor; - PhysicalSize::new(width.round() as _, height.round() as _) + let width = self.width.into() * dpi_factor; + let height = self.height.into() * dpi_factor; + PhysicalSize::new(width, height).cast() } -} -impl From<(f64, f64)> for LogicalSize { #[inline] - fn from((width, height): (f64, f64)) -> Self { - Self::new(width, height) - } -} - -impl From<(u32, u32)> for LogicalSize { - #[inline] - fn from((width, height): (u32, u32)) -> Self { - Self::new(width as f64, height as f64) + pub fn cast(&self) -> LogicalSize { + LogicalSize { + width: self.width.cast(), + height: self.height.cast(), + } } } -impl Into<(f64, f64)> for LogicalSize { - #[inline] - fn into(self) -> (f64, f64) { - (self.width, self.height) +impl From<(X, X)> for LogicalSize

{ + fn from((x, y): (X, X)) -> LogicalSize

{ + LogicalSize::new(x.cast(), y.cast()) } } -impl Into<(u32, u32)> for LogicalSize { - /// Note that this rounds instead of truncating. - #[inline] - fn into(self) -> (u32, u32) { - (self.width.round() as _, self.height.round() as _) +impl Into<(X, X)> for LogicalSize

{ + fn into(self: LogicalSize

) -> (X, X) { + (self.width.cast(), self.height.cast()) } } /// A size represented in physical pixels. #[derive(Debug, Copy, Clone, PartialEq, Eq, Hash)] #[cfg_attr(feature = "serde", derive(Serialize, Deserialize))] -pub struct PhysicalSize { - pub width: u32, - pub height: u32, +pub struct PhysicalSize

{ + pub width: P, + pub height: P, } -impl PhysicalSize { +impl

PhysicalSize

{ #[inline] - pub const fn new(width: u32, height: u32) -> Self { + pub const fn new(width: P, height: P) -> Self { PhysicalSize { width, height } } +} +impl PhysicalSize

{ #[inline] - pub fn from_logical>(logical: T, dpi_factor: f64) -> Self { + pub fn from_logical>, X: Pixel>(logical: T, dpi_factor: f64) -> Self { logical.into().to_physical(dpi_factor) } #[inline] - pub fn to_logical(&self, dpi_factor: f64) -> LogicalSize { + pub fn to_logical(&self, dpi_factor: f64) -> LogicalSize { assert!(validate_hidpi_factor(dpi_factor)); - let width = self.width as f64 / dpi_factor; - let height = self.height as f64 / dpi_factor; - LogicalSize::new(width, height) + let width = self.width.into() / dpi_factor; + let height = self.height.into() / dpi_factor; + LogicalSize::new(width, height).cast() } -} -impl From<(u32, u32)> for PhysicalSize { #[inline] - fn from((width, height): (u32, u32)) -> Self { - Self::new(width, height) + pub fn cast(&self) -> PhysicalSize { + PhysicalSize { + width: self.width.cast(), + height: self.height.cast(), + } } } -impl Into<(u32, u32)> for PhysicalSize { - /// Note that this rounds instead of truncating. - #[inline] - fn into(self) -> (u32, u32) { - (self.width, self.height) +impl From<(X, X)> for PhysicalSize

{ + fn from((x, y): (X, X)) -> PhysicalSize

{ + PhysicalSize::new(x.cast(), y.cast()) + } +} + +impl Into<(X, X)> for PhysicalSize

{ + fn into(self: Self) -> (X, X) { + (self.width.cast(), self.height.cast()) } } #[derive(Debug, Copy, Clone, PartialEq)] #[cfg_attr(feature = "serde", derive(Serialize, Deserialize))] pub enum Size { - Physical(PhysicalSize), - Logical(LogicalSize), + Physical(PhysicalSize), + Logical(LogicalSize), } impl Size { @@ -324,40 +364,40 @@ impl Size { size.into() } - pub fn to_logical(&self, dpi_factor: f64) -> LogicalSize { + pub fn to_logical(&self, dpi_factor: f64) -> LogicalSize

{ match *self { Size::Physical(size) => size.to_logical(dpi_factor), - Size::Logical(size) => size, + Size::Logical(size) => size.cast(), } } - pub fn to_physical(&self, dpi_factor: f64) -> PhysicalSize { + pub fn to_physical(&self, dpi_factor: f64) -> PhysicalSize

{ match *self { - Size::Physical(size) => size, + Size::Physical(size) => size.cast(), Size::Logical(size) => size.to_physical(dpi_factor), } } } -impl From for Size { +impl From> for Size { #[inline] - fn from(size: PhysicalSize) -> Size { - Size::Physical(size) + fn from(size: PhysicalSize

) -> Size { + Size::Physical(size.cast()) } } -impl From for Size { +impl From> for Size { #[inline] - fn from(size: LogicalSize) -> Size { - Size::Logical(size) + fn from(size: LogicalSize

) -> Size { + Size::Logical(size.cast()) } } #[derive(Debug, Copy, Clone, PartialEq)] #[cfg_attr(feature = "serde", derive(Serialize, Deserialize))] pub enum Position { - Physical(PhysicalPosition), - Logical(LogicalPosition), + Physical(PhysicalPosition), + Logical(LogicalPosition), } impl Position { @@ -365,31 +405,31 @@ impl Position { position.into() } - pub fn to_logical(&self, dpi_factor: f64) -> LogicalPosition { + pub fn to_logical(&self, dpi_factor: f64) -> LogicalPosition

{ match *self { Position::Physical(position) => position.to_logical(dpi_factor), - Position::Logical(position) => position, + Position::Logical(position) => position.cast(), } } - pub fn to_physical(&self, dpi_factor: f64) -> PhysicalPosition { + pub fn to_physical(&self, dpi_factor: f64) -> PhysicalPosition

{ match *self { - Position::Physical(position) => position, + Position::Physical(position) => position.cast(), Position::Logical(position) => position.to_physical(dpi_factor), } } } -impl From for Position { +impl From> for Position { #[inline] - fn from(position: PhysicalPosition) -> Position { - Position::Physical(position) + fn from(position: PhysicalPosition

) -> Position { + Position::Physical(position.cast()) } } -impl From for Position { +impl From> for Position { #[inline] - fn from(position: LogicalPosition) -> Position { - Position::Logical(position) + fn from(position: LogicalPosition

) -> Position { + Position::Logical(position.cast()) } } diff --git a/src/event.rs b/src/event.rs index 0a215b98ba..6277c7c549 100644 --- a/src/event.rs +++ b/src/event.rs @@ -126,10 +126,10 @@ pub enum StartCause { #[derive(Debug, PartialEq)] pub enum WindowEvent<'a> { /// The size of the window has changed. Contains the client area's new dimensions. - Resized(PhysicalSize), + Resized(PhysicalSize), /// The position of the window has changed. Contains the window's new position. - Moved(PhysicalPosition), + Moved(PhysicalPosition), /// The window has been requested to close. CloseRequested, @@ -186,7 +186,7 @@ pub enum WindowEvent<'a> { /// (x,y) coords in pixels relative to the top-left corner of the window. Because the range of this data is /// limited by the display area and it may have been transformed by the OS to implement effects such as cursor /// acceleration, it should not be used to implement non-cursor-like interactions such as 3D camera control. - position: PhysicalPosition, + position: PhysicalPosition, #[deprecated = "Deprecated in favor of DeviceEvent::ModifiersChanged"] modifiers: ModifiersState, }, @@ -252,7 +252,7 @@ pub enum WindowEvent<'a> { /// For more information about DPI in general, see the [`dpi`](dpi/index.html) module. HiDpiFactorChanged { hidpi_factor: f64, - new_inner_size: &'a mut Option, + new_inner_size: &'a mut Option>, }, /// The system window theme has changed. @@ -478,7 +478,7 @@ pub enum TouchPhase { pub struct Touch { pub device_id: DeviceId, pub phase: TouchPhase, - pub location: PhysicalPosition, + pub location: PhysicalPosition, /// Describes how hard the screen was pressed. May be `None` if the platform /// does not support pressure sensitivity. /// @@ -589,7 +589,7 @@ pub enum MouseScrollDelta { /// Scroll events are expressed as a PixelDelta if /// supported by the device (eg. a touchpad) and /// platform. - PixelDelta(LogicalPosition), + PixelDelta(LogicalPosition), } /// Symbolic name for a keyboard key. diff --git a/src/monitor.rs b/src/monitor.rs index 417596c379..c9d6e061ce 100644 --- a/src/monitor.rs +++ b/src/monitor.rs @@ -58,7 +58,7 @@ impl Ord for VideoMode { impl VideoMode { /// Returns the resolution of this video mode. #[inline] - pub fn size(&self) -> PhysicalSize { + pub fn size(&self) -> PhysicalSize { self.video_mode.size() } @@ -133,7 +133,7 @@ impl MonitorHandle { /// /// - **Web:** Always returns (0,0) #[inline] - pub fn size(&self) -> PhysicalSize { + pub fn size(&self) -> PhysicalSize { self.inner.size() } @@ -144,7 +144,7 @@ impl MonitorHandle { /// /// - **Web:** Always returns (0,0) #[inline] - pub fn position(&self) -> PhysicalPosition { + pub fn position(&self) -> PhysicalPosition { self.inner.position() } diff --git a/src/platform/macos.rs b/src/platform/macos.rs index 8101f0f7c3..cfedaab23f 100644 --- a/src/platform/macos.rs +++ b/src/platform/macos.rs @@ -128,7 +128,7 @@ pub trait WindowBuilderExtMacOS { /// Makes the window content appear behind the titlebar. fn with_fullsize_content_view(self, fullsize_content_view: bool) -> WindowBuilder; /// Build window with `resizeIncrements` property. Values must not be 0. - fn with_resize_increments(self, increments: LogicalSize) -> WindowBuilder; + fn with_resize_increments(self, increments: LogicalSize) -> WindowBuilder; fn with_disallow_hidpi(self, disallow_hidpi: bool) -> WindowBuilder; } @@ -179,7 +179,7 @@ impl WindowBuilderExtMacOS for WindowBuilder { } #[inline] - fn with_resize_increments(mut self, increments: LogicalSize) -> WindowBuilder { + fn with_resize_increments(mut self, increments: LogicalSize) -> WindowBuilder { self.platform_specific.resize_increments = Some(increments.into()); self } diff --git a/src/platform/unix.rs b/src/platform/unix.rs index 61921e1d82..fbf8a920a7 100644 --- a/src/platform/unix.rs +++ b/src/platform/unix.rs @@ -380,9 +380,9 @@ pub trait WindowBuilderExtUnix { /// Build window with `_GTK_THEME_VARIANT` hint set to the specified value. Currently only relevant on X11. fn with_gtk_theme_variant(self, variant: String) -> Self; /// Build window with resize increment hint. Only implemented on X11. - fn with_resize_increments(self, increments: LogicalSize) -> Self; + fn with_resize_increments(self, increments: LogicalSize) -> Self; /// Build window with base size hint. Only implemented on X11. - fn with_base_size(self, base_size: LogicalSize) -> Self; + fn with_base_size(self, base_size: LogicalSize) -> Self; /// Build window with a given application ID. It should match the `.desktop` file distributed with /// your program. Only relevant on Wayland. @@ -431,13 +431,13 @@ impl WindowBuilderExtUnix for WindowBuilder { } #[inline] - fn with_resize_increments(mut self, increments: LogicalSize) -> Self { + fn with_resize_increments(mut self, increments: LogicalSize) -> Self { self.platform_specific.resize_increments = Some(increments.into()); self } #[inline] - fn with_base_size(mut self, base_size: LogicalSize) -> Self { + fn with_base_size(mut self, base_size: LogicalSize) -> Self { self.platform_specific.base_size = Some(base_size.into()); self } diff --git a/src/platform_impl/android/mod.rs b/src/platform_impl/android/mod.rs index 4304e25482..11a140de6b 100644 --- a/src/platform_impl/android/mod.rs +++ b/src/platform_impl/android/mod.rs @@ -193,8 +193,8 @@ impl fmt::Debug for MonitorHandle { #[derive(Debug)] struct MonitorHandle { name: Option, - dimensions: PhysicalSize, - position: PhysicalPosition, + dimensions: PhysicalSize, + position: PhysicalPosition, hidpi_factor: f64, } @@ -216,7 +216,7 @@ impl MonitorHandle { } #[inline] - pub fn size(&self) -> PhysicalSize { + pub fn size(&self) -> PhysicalSize { unsafe { let window = android_glue::native_window(); ( @@ -228,7 +228,7 @@ impl MonitorHandle { } #[inline] - pub fn outer_position(&self) -> PhysicalPosition { + pub fn outer_position(&self) -> PhysicalPosition { // Android assumes single screen (0, 0).into() } @@ -283,29 +283,29 @@ impl Window { } #[inline] - pub fn outer_position(&self) -> Option { + pub fn outer_position(&self) -> Option> { // N/A None } #[inline] - pub fn inner_position(&self) -> Option { + pub fn inner_position(&self) -> Option> { // N/A None } #[inline] - pub fn set_outer_position(&self, _position: LogicalPosition) { + pub fn set_outer_position(&self, _position: LogicalPosition) { // N/A } #[inline] - pub fn set_min_inner_size(&self, _dimensions: Option) { + pub fn set_min_inner_size(&self, _dimensions: Option>) { // N/A } #[inline] - pub fn set_max_inner_size(&self, _dimensions: Option) { + pub fn set_max_inner_size(&self, _dimensions: Option>) { // N/A } @@ -315,7 +315,7 @@ impl Window { } #[inline] - pub fn inner_size(&self) -> Option { + pub fn inner_size(&self) -> Option> { if self.native_window.is_null() { None } else { @@ -326,12 +326,12 @@ impl Window { } #[inline] - pub fn outer_size(&self) -> Option { + pub fn outer_size(&self) -> Option> { self.inner_size() } #[inline] - pub fn set_inner_size(&self, _size: LogicalSize) { + pub fn set_inner_size(&self, _size: LogicalSize) { // N/A } @@ -356,7 +356,10 @@ impl Window { } #[inline] - pub fn set_cursor_position(&self, _position: LogicalPosition) -> Result<(), ExternalError> { + pub fn set_cursor_position( + &self, + _position: LogicalPosition, + ) -> Result<(), ExternalError> { Err(ExternalError::NotSupported(NotSupportedError::new())) } @@ -400,7 +403,7 @@ impl Window { } #[inline] - pub fn set_ime_position(&self, _spot: LogicalPosition) { + pub fn set_ime_position(&self, _spot: LogicalPosition) { // N/A } diff --git a/src/platform_impl/ios/app_state.rs b/src/platform_impl/ios/app_state.rs index 45b995e3a7..e295091640 100644 --- a/src/platform_impl/ios/app_state.rs +++ b/src/platform_impl/ios/app_state.rs @@ -806,7 +806,9 @@ pub unsafe fn handle_main_events_cleared() { let mut redraw_events: Vec> = this .main_events_cleared_transition() .into_iter() - .map(|window| EventWrapper::StaticEvent(Event::RedrawRequested(RootWindowId(window.into())))) + .map(|window| { + EventWrapper::StaticEvent(Event::RedrawRequested(RootWindowId(window.into()))) + }) .collect(); if !redraw_events.is_empty() { @@ -855,7 +857,7 @@ fn handle_event_proxy( fn handle_hidpi_proxy( event_handler: &mut Box, mut control_flow: ControlFlow, - suggested_size: LogicalSize, + suggested_size: LogicalSize, hidpi_factor: f64, window_id: id, ) { diff --git a/src/platform_impl/ios/event_loop.rs b/src/platform_impl/ios/event_loop.rs index 869005191c..b2987298d0 100644 --- a/src/platform_impl/ios/event_loop.rs +++ b/src/platform_impl/ios/event_loop.rs @@ -39,7 +39,7 @@ pub enum EventWrapper { pub enum EventProxy { HiDpiFactorChangedProxy { window_id: id, - suggested_size: LogicalSize, + suggested_size: LogicalSize, hidpi_factor: f64, }, } diff --git a/src/platform_impl/ios/ffi.rs b/src/platform_impl/ios/ffi.rs index c7cac81622..1c464b066a 100644 --- a/src/platform_impl/ios/ffi.rs +++ b/src/platform_impl/ios/ffi.rs @@ -43,7 +43,7 @@ pub struct CGSize { } impl CGSize { - pub fn new(size: LogicalSize) -> CGSize { + pub fn new(size: LogicalSize) -> CGSize { CGSize { width: size.width as _, height: size.height as _, diff --git a/src/platform_impl/ios/monitor.rs b/src/platform_impl/ios/monitor.rs index 71ad739083..e265954a69 100644 --- a/src/platform_impl/ios/monitor.rs +++ b/src/platform_impl/ios/monitor.rs @@ -88,7 +88,7 @@ impl VideoMode { } } - pub fn size(&self) -> PhysicalSize { + pub fn size(&self) -> PhysicalSize { self.size.into() } @@ -171,8 +171,8 @@ impl fmt::Debug for MonitorHandle { #[derive(Debug)] struct MonitorHandle { name: Option, - size: PhysicalSize, - position: PhysicalPosition, + size: PhysicalSize, + position: PhysicalPosition, hidpi_factor: f64, } @@ -216,14 +216,14 @@ impl Inner { } } - pub fn size(&self) -> PhysicalSize { + pub fn size(&self) -> PhysicalSize { unsafe { let bounds: CGRect = msg_send![self.ui_screen(), nativeBounds]; PhysicalSize::new(bounds.size.width as u32, bounds.size.height as u32) } } - pub fn position(&self) -> PhysicalPosition { + pub fn position(&self) -> PhysicalPosition { unsafe { let bounds: CGRect = msg_send![self.ui_screen(), nativeBounds]; (bounds.origin.x as f64, bounds.origin.y as f64).into() diff --git a/src/platform_impl/ios/view.rs b/src/platform_impl/ios/view.rs index e31a99ed65..a2419e9d05 100644 --- a/src/platform_impl/ios/view.rs +++ b/src/platform_impl/ios/view.rs @@ -103,8 +103,12 @@ unsafe fn get_view_class(root_view_class: &'static Class) -> &'static Class { let window: id = msg_send![object, window]; assert!(!window.is_null()); app_state::handle_nonuser_events( - std::iter::once(EventWrapper::StaticEvent(Event::RedrawRequested(RootWindowId(window.into())))) - .chain(std::iter::once(EventWrapper::StaticEvent(Event::RedrawEventsCleared))), + std::iter::once(EventWrapper::StaticEvent(Event::RedrawRequested( + RootWindowId(window.into()), + ))) + .chain(std::iter::once(EventWrapper::StaticEvent( + Event::RedrawEventsCleared, + ))), ); let superclass: &'static Class = msg_send![object, superclass]; let () = msg_send![super(object, superclass), drawRect: rect]; @@ -125,8 +129,8 @@ unsafe fn get_view_class(root_view_class: &'static Class) -> &'static Class { msg_send![object, convertRect:bounds toCoordinateSpace:screen_space]; let dpi_factor: CGFloat = msg_send![screen, scale]; let size = crate::dpi::LogicalSize { - width: screen_frame.size.width as _, - height: screen_frame.size.height as _, + width: screen_frame.size.width as f64, + height: screen_frame.size.height as f64, } .to_physical(dpi_factor.into()); app_state::handle_nonuser_event(EventWrapper::StaticEvent(Event::WindowEvent { @@ -508,7 +512,9 @@ pub unsafe fn create_window( let () = msg_send![uiscreen, setCurrentMode: video_mode.video_mode.screen_mode.0]; msg_send![window, setScreen:video_mode.monitor().ui_screen()] } - Some(Fullscreen::Borderless(ref monitor)) => msg_send![window, setScreen:monitor.ui_screen()], + Some(Fullscreen::Borderless(ref monitor)) => { + msg_send![window, setScreen:monitor.ui_screen()] + } None => (), } diff --git a/src/platform_impl/ios/window.rs b/src/platform_impl/ios/window.rs index f24e61c0d7..0fec4f75c9 100644 --- a/src/platform_impl/ios/window.rs +++ b/src/platform_impl/ios/window.rs @@ -76,24 +76,24 @@ impl Inner { } } - pub fn inner_position(&self) -> Result { + pub fn inner_position(&self) -> Result, NotSupportedError> { unsafe { let safe_area = self.safe_area_screen_space(); let position = LogicalPosition { - x: safe_area.origin.x as _, - y: safe_area.origin.y as _, + x: safe_area.origin.x as f64, + y: safe_area.origin.y as f64, }; let dpi_factor = self.hidpi_factor(); Ok(position.to_physical(dpi_factor)) } } - pub fn outer_position(&self) -> Result { + pub fn outer_position(&self) -> Result, NotSupportedError> { unsafe { let screen_frame = self.screen_frame(); let position = LogicalPosition { - x: screen_frame.origin.x as _, - y: screen_frame.origin.y as _, + x: screen_frame.origin.x as f64, + y: screen_frame.origin.y as f64, }; let dpi_factor = self.hidpi_factor(); Ok(position.to_physical(dpi_factor)) @@ -103,7 +103,7 @@ impl Inner { pub fn set_outer_position(&self, physical_position: Position) { unsafe { let dpi_factor = self.hidpi_factor(); - let position = physical_position.to_logical(dpi_factor); + let position = physical_position.to_logical::(dpi_factor); let screen_frame = self.screen_frame(); let new_screen_frame = CGRect { origin: CGPoint { @@ -117,25 +117,25 @@ impl Inner { } } - pub fn inner_size(&self) -> PhysicalSize { + pub fn inner_size(&self) -> PhysicalSize { unsafe { let dpi_factor = self.hidpi_factor(); let safe_area = self.safe_area_screen_space(); let size = LogicalSize { - width: safe_area.size.width as _, - height: safe_area.size.height as _, + width: safe_area.size.width as f64, + height: safe_area.size.height as f64, }; size.to_physical(dpi_factor) } } - pub fn outer_size(&self) -> PhysicalSize { + pub fn outer_size(&self) -> PhysicalSize { unsafe { let dpi_factor = self.hidpi_factor(); let screen_frame = self.screen_frame(); let size = LogicalSize { - width: screen_frame.size.width as _, - height: screen_frame.size.height as _, + width: screen_frame.size.width as f64, + height: screen_frame.size.height as f64, }; size.to_physical(dpi_factor) } @@ -356,7 +356,7 @@ impl Window { let frame = match window_attributes.inner_size { Some(dim) => { let dpi_factor = msg_send![screen, scale]; - let size = dim.to_logical(dpi_factor); + let size = dim.to_logical::(dpi_factor); CGRect { origin: screen_bounds.origin, size: CGSize { diff --git a/src/platform_impl/linux/mod.rs b/src/platform_impl/linux/mod.rs index 350e05f06e..cd7ed4a704 100644 --- a/src/platform_impl/linux/mod.rs +++ b/src/platform_impl/linux/mod.rs @@ -132,7 +132,7 @@ impl MonitorHandle { } #[inline] - pub fn size(&self) -> PhysicalSize { + pub fn size(&self) -> PhysicalSize { match self { &MonitorHandle::X(ref m) => m.size(), &MonitorHandle::Wayland(ref m) => m.size(), @@ -140,7 +140,7 @@ impl MonitorHandle { } #[inline] - pub fn position(&self) -> PhysicalPosition { + pub fn position(&self) -> PhysicalPosition { match self { &MonitorHandle::X(ref m) => m.position(), &MonitorHandle::Wayland(ref m) => m.position(), @@ -172,7 +172,7 @@ pub enum VideoMode { impl VideoMode { #[inline] - pub fn size(&self) -> PhysicalSize { + pub fn size(&self) -> PhysicalSize { match self { &VideoMode::X(ref m) => m.size(), &VideoMode::Wayland(ref m) => m.size(), @@ -246,7 +246,7 @@ impl Window { } #[inline] - pub fn outer_position(&self) -> Result { + pub fn outer_position(&self) -> Result, NotSupportedError> { match self { &Window::X(ref w) => w.outer_position(), &Window::Wayland(ref w) => w.outer_position(), @@ -254,7 +254,7 @@ impl Window { } #[inline] - pub fn inner_position(&self) -> Result { + pub fn inner_position(&self) -> Result, NotSupportedError> { match self { &Window::X(ref m) => m.inner_position(), &Window::Wayland(ref m) => m.inner_position(), @@ -270,7 +270,7 @@ impl Window { } #[inline] - pub fn inner_size(&self) -> PhysicalSize { + pub fn inner_size(&self) -> PhysicalSize { match self { &Window::X(ref w) => w.inner_size(), &Window::Wayland(ref w) => w.inner_size(), @@ -278,7 +278,7 @@ impl Window { } #[inline] - pub fn outer_size(&self) -> PhysicalSize { + pub fn outer_size(&self) -> PhysicalSize { match self { &Window::X(ref w) => w.outer_size(), &Window::Wayland(ref w) => w.outer_size(), diff --git a/src/platform_impl/linux/wayland/event_loop.rs b/src/platform_impl/linux/wayland/event_loop.rs index c538e77f8b..72d7815669 100644 --- a/src/platform_impl/linux/wayland/event_loop.rs +++ b/src/platform_impl/linux/wayland/event_loop.rs @@ -729,7 +729,7 @@ impl EventLoop { if let Some(dpi) = window.new_dpi { let dpi = dpi as f64; - let logical_size = LogicalSize::from(*window.size); + let logical_size = LogicalSize::::from(*window.size); let mut new_inner_size = Some(logical_size.to_physical(dpi)); callback(Event::WindowEvent { @@ -741,7 +741,7 @@ impl EventLoop { }); if let Some(new_size) = new_inner_size { - let (w, h) = new_size.to_logical(dpi).into(); + let (w, h) = new_size.to_logical::(dpi).into(); frame.resize(w, h); *window.size = (w, h); } @@ -947,7 +947,7 @@ pub struct VideoMode { impl VideoMode { #[inline] - pub fn size(&self) -> PhysicalSize { + pub fn size(&self) -> PhysicalSize { self.size.into() } @@ -1007,8 +1007,8 @@ impl fmt::Debug for MonitorHandle { struct MonitorHandle { name: Option, native_identifier: u32, - size: PhysicalSize, - position: PhysicalPosition, + size: PhysicalSize, + position: PhysicalPosition, hidpi_factor: i32, } @@ -1036,7 +1036,7 @@ impl MonitorHandle { self.mgr.with_info(&self.proxy, |id, _| id).unwrap_or(0) } - pub fn size(&self) -> PhysicalSize { + pub fn size(&self) -> PhysicalSize { match self.mgr.with_info(&self.proxy, |_, info| { info.modes .iter() @@ -1049,7 +1049,7 @@ impl MonitorHandle { .into() } - pub fn position(&self) -> PhysicalPosition { + pub fn position(&self) -> PhysicalPosition { self.mgr .with_info(&self.proxy, |_, info| info.location) .unwrap_or((0, 0)) diff --git a/src/platform_impl/linux/wayland/window.rs b/src/platform_impl/linux/wayland/window.rs index 2c509bbfa7..be2d0df938 100644 --- a/src/platform_impl/linux/wayland/window.rs +++ b/src/platform_impl/linux/wayland/window.rs @@ -60,7 +60,7 @@ impl Window { let dpi = get_dpi_factor(&surface) as f64; let (width, height) = attributes .inner_size - .map(|size| size.to_logical(dpi).into()) + .map(|size| size.to_logical::(dpi).into()) .unwrap_or((800, 600)); // Create the window @@ -147,12 +147,12 @@ impl Window { frame.set_min_size( attributes .min_inner_size - .map(|size| size.to_logical(dpi).into()), + .map(|size| size.to_logical::(dpi).into()), ); frame.set_max_size( attributes .max_inner_size - .map(|size| size.to_logical(dpi).into()), + .map(|size| size.to_logical::(dpi).into()), ); let kill_switch = Arc::new(Mutex::new(false)); @@ -206,12 +206,12 @@ impl Window { } #[inline] - pub fn outer_position(&self) -> Result { + pub fn outer_position(&self) -> Result, NotSupportedError> { Err(NotSupportedError::new()) } #[inline] - pub fn inner_position(&self) -> Result { + pub fn inner_position(&self) -> Result, NotSupportedError> { Err(NotSupportedError::new()) } @@ -220,9 +220,9 @@ impl Window { // Not possible with wayland } - pub fn inner_size(&self) -> PhysicalSize { + pub fn inner_size(&self) -> PhysicalSize { let dpi = self.hidpi_factor() as f64; - let size = LogicalSize::from(*self.size.lock().unwrap()); + let size = LogicalSize::::from(*self.size.lock().unwrap()); size.to_physical(dpi) } @@ -231,11 +231,11 @@ impl Window { } #[inline] - pub fn outer_size(&self) -> PhysicalSize { + pub fn outer_size(&self) -> PhysicalSize { let dpi = self.hidpi_factor() as f64; let (w, h) = self.size.lock().unwrap().clone(); // let (w, h) = super::wayland_window::add_borders(w as i32, h as i32); - let size = LogicalSize::from((w, h)); + let size = LogicalSize::::from((w, h)); size.to_physical(dpi) } @@ -243,7 +243,7 @@ impl Window { // NOTE: This will only resize the borders, the contents must be updated by the user pub fn set_inner_size(&self, size: Size) { let dpi = self.hidpi_factor() as f64; - let (w, h) = size.to_logical(dpi).into(); + let (w, h) = size.to_logical::(dpi).into(); self.frame.lock().unwrap().resize(w, h); *(self.size.lock().unwrap()) = (w, h); } @@ -254,7 +254,7 @@ impl Window { self.frame .lock() .unwrap() - .set_min_size(dimensions.map(|dim| dim.to_logical(dpi).into())); + .set_min_size(dimensions.map(|dim| dim.to_logical::(dpi).into())); } #[inline] @@ -263,7 +263,7 @@ impl Window { self.frame .lock() .unwrap() - .set_max_size(dimensions.map(|dim| dim.to_logical(dpi).into())); + .set_max_size(dimensions.map(|dim| dim.to_logical::(dpi).into())); } #[inline] diff --git a/src/platform_impl/linux/x11/event_processor.rs b/src/platform_impl/linux/x11/event_processor.rs index 884a7ea149..33a5e14902 100644 --- a/src/platform_impl/linux/x11/event_processor.rs +++ b/src/platform_impl/linux/x11/event_processor.rs @@ -707,7 +707,7 @@ impl EventProcessor { }); if cursor_moved == Some(true) { let position = - PhysicalPosition::new(xev.event_x as f64, xev.event_y as f64); + PhysicalPosition::new(xev.event_x as i32, xev.event_y as i32); callback(Event::WindowEvent { window_id, @@ -814,7 +814,7 @@ impl EventProcessor { }); let position = - PhysicalPosition::new(xev.event_x as f64, xev.event_y as f64); + PhysicalPosition::new(xev.event_x as i32, xev.event_y as i32); // The mods field on this event isn't actually populated, so query the // pointer device. In the future, we can likely remove this round-trip by @@ -883,7 +883,7 @@ impl EventProcessor { .unwrap_or(2); let position = - PhysicalPosition::new(xev.event_x as f64, xev.event_y as f64); + PhysicalPosition::new(xev.event_x as i32, xev.event_y as i32); callback(Event::WindowEvent { window_id, @@ -941,7 +941,7 @@ impl EventProcessor { window_id, event: WindowEvent::CursorMoved { device_id: mkdid(util::VIRTUAL_CORE_POINTER), - position: location, + position: location.cast(), modifiers, }, }); diff --git a/src/platform_impl/linux/x11/monitor.rs b/src/platform_impl/linux/x11/monitor.rs index 2612643e2a..98c8022b40 100644 --- a/src/platform_impl/linux/x11/monitor.rs +++ b/src/platform_impl/linux/x11/monitor.rs @@ -38,7 +38,7 @@ pub struct VideoMode { impl VideoMode { #[inline] - pub fn size(&self) -> PhysicalSize { + pub fn size(&self) -> PhysicalSize { self.size.into() } @@ -157,11 +157,11 @@ impl MonitorHandle { self.id as u32 } - pub fn size(&self) -> PhysicalSize { + pub fn size(&self) -> PhysicalSize { self.dimensions.into() } - pub fn position(&self) -> PhysicalPosition { + pub fn position(&self) -> PhysicalPosition { self.position.into() } diff --git a/src/platform_impl/linux/x11/util/geometry.rs b/src/platform_impl/linux/x11/util/geometry.rs index 6b59d13a2a..149418e320 100644 --- a/src/platform_impl/linux/x11/util/geometry.rs +++ b/src/platform_impl/linux/x11/util/geometry.rs @@ -137,9 +137,9 @@ impl FrameExtentsHeuristic { pub fn inner_pos_to_outer_logical( &self, - mut logical: LogicalPosition, + mut logical: LogicalPosition, factor: f64, - ) -> LogicalPosition { + ) -> LogicalPosition { use self::FrameExtentsHeuristicPath::*; if self.heuristic_path != UnsupportedBordered { let frame_extents = self.frame_extents.as_logical(factor); @@ -166,9 +166,9 @@ impl FrameExtentsHeuristic { pub fn inner_size_to_outer_logical( &self, - mut logical: LogicalSize, + mut logical: LogicalSize, factor: f64, - ) -> LogicalSize { + ) -> LogicalSize { let frame_extents = self.frame_extents.as_logical(factor); logical.width += frame_extents.left + frame_extents.right; logical.height += frame_extents.top + frame_extents.bottom; diff --git a/src/platform_impl/linux/x11/window.rs b/src/platform_impl/linux/x11/window.rs index 0a72875b09..9fd7d89198 100644 --- a/src/platform_impl/linux/x11/window.rs +++ b/src/platform_impl/linux/x11/window.rs @@ -138,17 +138,17 @@ impl UnownedWindow { let max_inner_size: Option<(u32, u32)> = window_attrs .max_inner_size - .map(|size| size.to_physical(dpi_factor).into()); + .map(|size| size.to_physical::(dpi_factor).into()); let min_inner_size: Option<(u32, u32)> = window_attrs .min_inner_size - .map(|size| size.to_physical(dpi_factor).into()); + .map(|size| size.to_physical::(dpi_factor).into()); let dimensions = { // x11 only applies constraints when the window is actively resized // by the user, so we have to manually apply the initial constraints let mut dimensions: (u32, u32) = window_attrs .inner_size - .map(|size| size.to_physical(dpi_factor)) + .map(|size| size.to_physical::(dpi_factor)) .or_else(|| Some((800, 600).into())) .map(Into::into) .unwrap(); @@ -320,10 +320,10 @@ impl UnownedWindow { { let mut min_inner_size = window_attrs .min_inner_size - .map(|size| size.to_physical(dpi_factor)); + .map(|size| size.to_physical::(dpi_factor)); let mut max_inner_size = window_attrs .max_inner_size - .map(|size| size.to_physical(dpi_factor)); + .map(|size| size.to_physical::(dpi_factor)); if !window_attrs.resizable { if util::wm_name_is_one_of(&["Xfwm4"]) { warn!("To avoid a WM bug, disabling resizing has no effect on Xfwm4"); @@ -941,7 +941,7 @@ impl UnownedWindow { } #[inline] - pub fn outer_position(&self) -> Result { + pub fn outer_position(&self) -> Result, NotSupportedError> { let extents = (*self.shared_state.lock()).frame_extents.clone(); if let Some(extents) = extents { let (x, y) = self.inner_position_physical(); @@ -962,7 +962,7 @@ impl UnownedWindow { } #[inline] - pub fn inner_position(&self) -> Result { + pub fn inner_position(&self) -> Result, NotSupportedError> { Ok(self.inner_position_physical().into()) } @@ -993,7 +993,7 @@ impl UnownedWindow { #[inline] pub fn set_outer_position(&self, position: Position) { - let (x, y) = position.to_physical(self.hidpi_factor()).into(); + let (x, y) = position.to_physical::(self.hidpi_factor()).into(); self.set_position_physical(x, y); } @@ -1007,12 +1007,12 @@ impl UnownedWindow { } #[inline] - pub fn inner_size(&self) -> PhysicalSize { + pub fn inner_size(&self) -> PhysicalSize { self.inner_size_physical().into() } #[inline] - pub fn outer_size(&self) -> PhysicalSize { + pub fn outer_size(&self) -> PhysicalSize { let extents = self.shared_state.lock().frame_extents.clone(); if let Some(extents) = extents { let (width, height) = self.inner_size_physical(); @@ -1039,7 +1039,7 @@ impl UnownedWindow { #[inline] pub fn set_inner_size(&self, size: Size) { let dpi_factor = self.hidpi_factor(); - let (width, height) = size.to_physical(dpi_factor).into(); + let (width, height) = size.to_physical::(dpi_factor).into(); self.set_inner_size_physical(width, height); } @@ -1063,7 +1063,7 @@ impl UnownedWindow { pub fn set_min_inner_size(&self, dimensions: Option) { self.shared_state.lock().min_inner_size = dimensions; let physical_dimensions = - dimensions.map(|dimensions| dimensions.to_physical(self.hidpi_factor()).into()); + dimensions.map(|dimensions| dimensions.to_physical::(self.hidpi_factor()).into()); self.set_min_inner_size_physical(physical_dimensions); } @@ -1076,7 +1076,7 @@ impl UnownedWindow { pub fn set_max_inner_size(&self, dimensions: Option) { self.shared_state.lock().max_inner_size = dimensions; let physical_dimensions = - dimensions.map(|dimensions| dimensions.to_physical(self.hidpi_factor()).into()); + dimensions.map(|dimensions| dimensions.to_physical::(self.hidpi_factor()).into()); self.set_max_inner_size_physical(physical_dimensions); } @@ -1135,10 +1135,10 @@ impl UnownedWindow { let dpi_factor = self.hidpi_factor(); let min_inner_size = min_size - .map(|size| size.to_physical(dpi_factor)) + .map(|size| size.to_physical::(dpi_factor)) .map(Into::into); let max_inner_size = max_size - .map(|size| size.to_physical(dpi_factor)) + .map(|size| size.to_physical::(dpi_factor)) .map(Into::into); self.update_normal_hints(|normal_hints| { normal_hints.set_min_size(min_inner_size); @@ -1274,7 +1274,7 @@ impl UnownedWindow { #[inline] pub fn set_cursor_position(&self, position: Position) -> Result<(), ExternalError> { - let (x, y) = position.to_physical(self.hidpi_factor()).into(); + let (x, y) = position.to_physical::(self.hidpi_factor()).into(); self.set_cursor_position_physical(x, y) } @@ -1287,7 +1287,7 @@ impl UnownedWindow { #[inline] pub fn set_ime_position(&self, spot: Position) { - let (x, y) = spot.to_physical(self.hidpi_factor()).into(); + let (x, y) = spot.to_physical::(self.hidpi_factor()).into(); self.set_ime_position_physical(x, y); } diff --git a/src/platform_impl/macos/app_state.rs b/src/platform_impl/macos/app_state.rs index 618be00233..7d6b92b2fc 100644 --- a/src/platform_impl/macos/app_state.rs +++ b/src/platform_impl/macos/app_state.rs @@ -185,7 +185,7 @@ impl Handler { &self, callback: &mut Box, ns_window: IdRef, - suggested_size: LogicalSize, + suggested_size: LogicalSize, hidpi_factor: f64, ) { let size = suggested_size.to_physical(hidpi_factor); diff --git a/src/platform_impl/macos/event.rs b/src/platform_impl/macos/event.rs index bbf111645b..ef33ba234f 100644 --- a/src/platform_impl/macos/event.rs +++ b/src/platform_impl/macos/event.rs @@ -24,7 +24,7 @@ pub enum EventWrapper { pub enum EventProxy { HiDpiFactorChangedProxy { ns_window: IdRef, - suggested_size: LogicalSize, + suggested_size: LogicalSize, hidpi_factor: f64, }, } diff --git a/src/platform_impl/macos/monitor.rs b/src/platform_impl/macos/monitor.rs index 976f6c9048..11a0aa2b5a 100644 --- a/src/platform_impl/macos/monitor.rs +++ b/src/platform_impl/macos/monitor.rs @@ -84,7 +84,7 @@ impl Clone for NativeDisplayMode { } impl VideoMode { - pub fn size(&self) -> PhysicalSize { + pub fn size(&self) -> PhysicalSize { self.size.into() } @@ -166,8 +166,8 @@ impl fmt::Debug for MonitorHandle { struct MonitorHandle { name: Option, native_identifier: u32, - size: PhysicalSize, - position: PhysicalPosition, + size: PhysicalSize, + position: PhysicalPosition, hidpi_factor: f64, } @@ -199,18 +199,18 @@ impl MonitorHandle { self.0 } - pub fn size(&self) -> PhysicalSize { + pub fn size(&self) -> PhysicalSize { let MonitorHandle(display_id) = *self; let display = CGDisplay::new(display_id); let height = display.pixels_high(); let width = display.pixels_wide(); - PhysicalSize::from_logical((width as f64, height as f64), self.hidpi_factor()) + PhysicalSize::from_logical::<_, f64>((width as f64, height as f64), self.hidpi_factor()) } #[inline] - pub fn position(&self) -> PhysicalPosition { + pub fn position(&self) -> PhysicalPosition { let bounds = unsafe { CGDisplayBounds(self.native_identifier()) }; - PhysicalPosition::from_logical( + PhysicalPosition::from_logical::<_, f64>( (bounds.origin.x as f64, bounds.origin.y as f64), self.hidpi_factor(), ) diff --git a/src/platform_impl/macos/util/async.rs b/src/platform_impl/macos/util/async.rs index c2d117b61c..8aa8db9a27 100644 --- a/src/platform_impl/macos/util/async.rs +++ b/src/platform_impl/macos/util/async.rs @@ -73,10 +73,10 @@ pub unsafe fn set_style_mask_sync(ns_window: id, ns_view: id, mask: NSWindowStyl struct SetContentSizeData { ns_window: id, - size: LogicalSize, + size: LogicalSize, } impl SetContentSizeData { - fn new_ptr(ns_window: id, size: LogicalSize) -> *mut Self { + fn new_ptr(ns_window: id, size: LogicalSize) -> *mut Self { Box::into_raw(Box::new(SetContentSizeData { ns_window, size })) } } @@ -98,7 +98,7 @@ extern "C" fn set_content_size_callback(context: *mut c_void) { } // `setContentSize:` isn't thread-safe either, though it doesn't log any errors // and just fails silently. Anyway, GCD to the rescue! -pub unsafe fn set_content_size_async(ns_window: id, size: LogicalSize) { +pub unsafe fn set_content_size_async(ns_window: id, size: LogicalSize) { let context = SetContentSizeData::new_ptr(ns_window, size); dispatch_async_f( dispatch_get_main_queue(), diff --git a/src/platform_impl/macos/window.rs b/src/platform_impl/macos/window.rs index 4ed62756be..5c3547a439 100644 --- a/src/platform_impl/macos/window.rs +++ b/src/platform_impl/macos/window.rs @@ -68,7 +68,7 @@ pub struct PlatformSpecificWindowBuilderAttributes { pub titlebar_hidden: bool, pub titlebar_buttons_hidden: bool, pub fullsize_content_view: bool, - pub resize_increments: Option, + pub resize_increments: Option>, pub disallow_hidpi: bool, } @@ -299,7 +299,7 @@ pub struct UnownedWindow { pub shared_state: Arc>, decorations: AtomicBool, cursor_state: Weak>, - pub inner_rect: Option, + pub inner_rect: Option>, } unsafe impl Send for UnownedWindow {} @@ -440,7 +440,7 @@ impl UnownedWindow { AppState::queue_redraw(RootWindowId(self.id())); } - pub fn outer_position(&self) -> Result { + pub fn outer_position(&self) -> Result, NotSupportedError> { let frame_rect = unsafe { NSWindow::frame(*self.ns_window) }; let position = LogicalPosition::new( frame_rect.origin.x as f64, @@ -450,7 +450,7 @@ impl UnownedWindow { Ok(position.to_physical(dpi_factor)) } - pub fn inner_position(&self) -> Result { + pub fn inner_position(&self) -> Result, NotSupportedError> { let content_rect = unsafe { NSWindow::contentRectForFrameRect_(*self.ns_window, NSWindow::frame(*self.ns_window)) }; @@ -480,18 +480,18 @@ impl UnownedWindow { } #[inline] - pub fn inner_size(&self) -> PhysicalSize { + pub fn inner_size(&self) -> PhysicalSize { let view_frame = unsafe { NSView::frame(*self.ns_view) }; - let logical: LogicalSize = + let logical: LogicalSize = (view_frame.size.width as f64, view_frame.size.height as f64).into(); let dpi_factor = self.hidpi_factor(); logical.to_physical(dpi_factor) } #[inline] - pub fn outer_size(&self) -> PhysicalSize { + pub fn outer_size(&self) -> PhysicalSize { let view_frame = unsafe { NSWindow::frame(*self.ns_window) }; - let logical: LogicalSize = + let logical: LogicalSize = (view_frame.size.width as f64, view_frame.size.height as f64).into(); let dpi_factor = self.hidpi_factor(); logical.to_physical(dpi_factor) @@ -591,11 +591,11 @@ impl UnownedWindow { pub fn set_cursor_position(&self, cursor_position: Position) -> Result<(), ExternalError> { let physical_window_position = self.inner_position().unwrap(); let dpi_factor = self.hidpi_factor(); - let window_position = physical_window_position.to_logical(dpi_factor); - let logical_cursor_position = cursor_position.to_logical(dpi_factor); + let window_position = physical_window_position.to_logical::(dpi_factor); + let logical_cursor_position = cursor_position.to_logical::(dpi_factor); let point = appkit::CGPoint { - x: (logical_cursor_position.x + window_position.x) as CGFloat, - y: (logical_cursor_position.y + window_position.y) as CGFloat, + x: logical_cursor_position.x + window_position.x, + y: logical_cursor_position.y + window_position.y, }; CGDisplay::warp_mouse_cursor_position(point) .map_err(|e| ExternalError::Os(os_error!(OsError::CGError(e))))?; @@ -1095,7 +1095,7 @@ impl Drop for UnownedWindow { } } -unsafe fn set_min_inner_size(window: V, mut min_size: LogicalSize) { +unsafe fn set_min_inner_size(window: V, mut min_size: LogicalSize) { let mut current_rect = NSWindow::frame(window); let content_rect = NSWindow::contentRectForFrameRect_(window, NSWindow::frame(window)); // Convert from client area size to window size @@ -1119,7 +1119,7 @@ unsafe fn set_min_inner_size(window: V, mut min_size: Logica } } -unsafe fn set_max_inner_size(window: V, mut max_size: LogicalSize) { +unsafe fn set_max_inner_size(window: V, mut max_size: LogicalSize) { let mut current_rect = NSWindow::frame(window); let content_rect = NSWindow::contentRectForFrameRect_(window, NSWindow::frame(window)); // Convert from client area size to window size diff --git a/src/platform_impl/macos/window_delegate.rs b/src/platform_impl/macos/window_delegate.rs index d4ac37b54d..6ffe88c3a4 100644 --- a/src/platform_impl/macos/window_delegate.rs +++ b/src/platform_impl/macos/window_delegate.rs @@ -118,7 +118,7 @@ impl WindowDelegateState { (unsafe { NSWindow::backingScaleFactor(*self.ns_window) }) as f64 } - fn view_size(&self) -> LogicalSize { + fn view_size(&self) -> LogicalSize { let ns_size = unsafe { NSView::frame(*self.ns_view).size }; LogicalSize::new(ns_size.width as f64, ns_size.height as f64) } diff --git a/src/platform_impl/web/monitor.rs b/src/platform_impl/web/monitor.rs index 4889c88db9..9442d3b87f 100644 --- a/src/platform_impl/web/monitor.rs +++ b/src/platform_impl/web/monitor.rs @@ -9,7 +9,7 @@ impl Handle { 1.0 } - pub fn position(&self) -> PhysicalPosition { + pub fn position(&self) -> PhysicalPosition { PhysicalPosition { x: 0.0, y: 0.0 } } @@ -17,7 +17,7 @@ impl Handle { None } - pub fn size(&self) -> PhysicalSize { + pub fn size(&self) -> PhysicalSize { PhysicalSize { width: 0.0, height: 0.0, @@ -33,7 +33,7 @@ impl Handle { pub struct Mode; impl Mode { - pub fn size(&self) -> PhysicalSize { + pub fn size(&self) -> PhysicalSize { unimplemented!(); } diff --git a/src/platform_impl/web/stdweb/canvas.rs b/src/platform_impl/web/stdweb/canvas.rs index a0ff5af2e8..07ba42b54e 100644 --- a/src/platform_impl/web/stdweb/canvas.rs +++ b/src/platform_impl/web/stdweb/canvas.rs @@ -96,7 +96,7 @@ impl Canvas { self.raw.height() as f64 } - pub fn set_size(&self, size: LogicalSize) { + pub fn set_size(&self, size: LogicalSize) { self.raw.set_width(size.width as u32); self.raw.set_height(size.height as u32); } diff --git a/src/platform_impl/web/stdweb/event.rs b/src/platform_impl/web/stdweb/event.rs index 223ef69325..8c534dc053 100644 --- a/src/platform_impl/web/stdweb/event.rs +++ b/src/platform_impl/web/stdweb/event.rs @@ -23,7 +23,7 @@ pub fn mouse_modifiers(event: &impl IMouseEvent) -> ModifiersState { m } -pub fn mouse_position(event: &impl IMouseEvent) -> LogicalPosition { +pub fn mouse_position(event: &impl IMouseEvent) -> LogicalPosition { LogicalPosition { x: event.offset_x() as f64, y: event.offset_y() as f64, diff --git a/src/platform_impl/web/stdweb/mod.rs b/src/platform_impl/web/stdweb/mod.rs index d49dbf02ac..0ac4b1725c 100644 --- a/src/platform_impl/web/stdweb/mod.rs +++ b/src/platform_impl/web/stdweb/mod.rs @@ -33,7 +33,7 @@ impl WindowExtStdweb for Window { } } -pub fn window_size() -> LogicalSize { +pub fn window_size() -> LogicalSize { let window = window(); let width = window.inner_width() as f64; let height = window.inner_height() as f64; diff --git a/src/platform_impl/web/web_sys/canvas.rs b/src/platform_impl/web/web_sys/canvas.rs index 0543055eb6..c21b8cbd66 100644 --- a/src/platform_impl/web/web_sys/canvas.rs +++ b/src/platform_impl/web/web_sys/canvas.rs @@ -94,7 +94,7 @@ impl Canvas { self.raw.height() as f64 } - pub fn set_size(&self, size: LogicalSize) { + pub fn set_size(&self, size: LogicalSize) { self.raw.set_width(size.width as u32); self.raw.set_height(size.height as u32); } diff --git a/src/platform_impl/web/web_sys/event.rs b/src/platform_impl/web/web_sys/event.rs index 9a11f3bfa1..1c1bdba41c 100644 --- a/src/platform_impl/web/web_sys/event.rs +++ b/src/platform_impl/web/web_sys/event.rs @@ -22,7 +22,7 @@ pub fn mouse_modifiers(event: &MouseEvent) -> ModifiersState { m } -pub fn mouse_position(event: &MouseEvent) -> LogicalPosition { +pub fn mouse_position(event: &MouseEvent) -> LogicalPosition { LogicalPosition { x: event.offset_x() as f64, y: event.offset_y() as f64, diff --git a/src/platform_impl/web/web_sys/mod.rs b/src/platform_impl/web/web_sys/mod.rs index 205519d1ea..4424a03bcb 100644 --- a/src/platform_impl/web/web_sys/mod.rs +++ b/src/platform_impl/web/web_sys/mod.rs @@ -40,7 +40,7 @@ impl WindowExtWebSys for Window { } } -pub fn window_size() -> LogicalSize { +pub fn window_size() -> LogicalSize { let window = web_sys::window().expect("Failed to obtain window"); let width = window .inner_width() diff --git a/src/platform_impl/web/window.rs b/src/platform_impl/web/window.rs index c14df77f4b..d63086aab1 100644 --- a/src/platform_impl/web/window.rs +++ b/src/platform_impl/web/window.rs @@ -15,7 +15,7 @@ use std::collections::VecDeque; pub struct Window { canvas: backend::Canvas, previous_pointer: RefCell<&'static str>, - position: RefCell, + position: RefCell>, id: Id, register_redraw_request: Box, } @@ -72,17 +72,17 @@ impl Window { (self.register_redraw_request)(); } - pub fn outer_position(&self) -> Result { + pub fn outer_position(&self) -> Result, NotSupportedError> { let (x, y) = self.canvas.position(); Ok(LogicalPosition { x, y }) } - pub fn inner_position(&self) -> Result { + pub fn inner_position(&self) -> Result, NotSupportedError> { Ok(*self.position.borrow()) } - pub fn set_outer_position(&self, position: LogicalPosition) { + pub fn set_outer_position(&self, position: LogicalPosition) { *self.position.borrow_mut() = position; self.canvas.set_attribute("position", "fixed"); @@ -91,7 +91,7 @@ impl Window { } #[inline] - pub fn inner_size(&self) -> LogicalSize { + pub fn inner_size(&self) -> LogicalSize { LogicalSize { width: self.canvas.width() as f64, height: self.canvas.height() as f64, @@ -99,7 +99,7 @@ impl Window { } #[inline] - pub fn outer_size(&self) -> LogicalSize { + pub fn outer_size(&self) -> LogicalSize { LogicalSize { width: self.canvas.width() as f64, height: self.canvas.height() as f64, @@ -107,17 +107,17 @@ impl Window { } #[inline] - pub fn set_inner_size(&self, size: LogicalSize) { + pub fn set_inner_size(&self, size: LogicalSize) { self.canvas.set_size(size); } #[inline] - pub fn set_min_inner_size(&self, _dimensions: Option) { + pub fn set_min_inner_size(&self, _dimensions: Option>) { // Intentionally a no-op: users can't resize canvas elements } #[inline] - pub fn set_max_inner_size(&self, _dimensions: Option) { + pub fn set_max_inner_size(&self, _dimensions: Option>) { // Intentionally a no-op: users can't resize canvas elements } @@ -178,7 +178,10 @@ impl Window { } #[inline] - pub fn set_cursor_position(&self, _position: LogicalPosition) -> Result<(), ExternalError> { + pub fn set_cursor_position( + &self, + _position: LogicalPosition, + ) -> Result<(), ExternalError> { // Intentionally a no-op, as the web does not support setting cursor positions Ok(()) } @@ -243,7 +246,7 @@ impl Window { } #[inline] - pub fn set_ime_position(&self, _position: LogicalPosition) { + pub fn set_ime_position(&self, _position: LogicalPosition) { // Currently a no-op as it does not seem there is good support for this on web } diff --git a/src/platform_impl/windows/event_loop.rs b/src/platform_impl/windows/event_loop.rs index 0568aa8c94..7783c6790b 100644 --- a/src/platform_impl/windows/event_loop.rs +++ b/src/platform_impl/windows/event_loop.rs @@ -707,7 +707,7 @@ unsafe extern "system" fn public_window_callback( let windowpos = lparam as *const winuser::WINDOWPOS; if (*windowpos).flags & winuser::SWP_NOMOVE != winuser::SWP_NOMOVE { let physical_position = - PhysicalPosition::new((*windowpos).x as f64, (*windowpos).y as f64); + PhysicalPosition::new((*windowpos).x as u32, (*windowpos).y as u32); subclass_input.send_event(Event::WindowEvent { window_id: RootWindowId(WindowId(window)), event: Moved(physical_position), @@ -829,8 +829,8 @@ unsafe extern "system" fn public_window_callback( }); } - let x = windowsx::GET_X_LPARAM(lparam) as f64; - let y = windowsx::GET_Y_LPARAM(lparam) as f64; + let x = windowsx::GET_X_LPARAM(lparam) as i32; + let y = windowsx::GET_Y_LPARAM(lparam) as i32; let position = PhysicalPosition::new(x, y); subclass_input.send_event(Event::WindowEvent { @@ -1489,8 +1489,8 @@ unsafe extern "system" fn public_window_callback( // We calculate our own size because the default suggested rect doesn't do a great job // of preserving the window's logical size. let suggested_physical_inner_size = old_physical_inner_size - .to_logical(old_dpi_factor) - .to_physical(new_dpi_factor); + .to_logical::(old_dpi_factor) + .to_physical::(new_dpi_factor); // `allow_resize` prevents us from re-applying DPI adjustment to the restored size after // exiting fullscreen (the restored size is already DPI adjusted). diff --git a/src/platform_impl/windows/monitor.rs b/src/platform_impl/windows/monitor.rs index e87a48c600..0b506dc3ba 100644 --- a/src/platform_impl/windows/monitor.rs +++ b/src/platform_impl/windows/monitor.rs @@ -62,7 +62,7 @@ impl std::fmt::Debug for VideoMode { } impl VideoMode { - pub fn size(&self) -> PhysicalSize { + pub fn size(&self) -> PhysicalSize { self.size.into() } @@ -185,7 +185,7 @@ impl MonitorHandle { } #[inline] - pub fn size(&self) -> PhysicalSize { + pub fn size(&self) -> PhysicalSize { let monitor_info = get_monitor_info(self.0).unwrap(); PhysicalSize { width: (monitor_info.rcMonitor.right - monitor_info.rcMonitor.left) as u32, @@ -194,11 +194,11 @@ impl MonitorHandle { } #[inline] - pub fn position(&self) -> PhysicalPosition { + pub fn position(&self) -> PhysicalPosition { let monitor_info = get_monitor_info(self.0).unwrap(); PhysicalPosition { - x: monitor_info.rcMonitor.left as f64, - y: monitor_info.rcMonitor.top as f64, + x: monitor_info.rcMonitor.left, + y: monitor_info.rcMonitor.top, } } diff --git a/src/platform_impl/windows/util.rs b/src/platform_impl/windows/util.rs index 36fc225368..1b9845038a 100644 --- a/src/platform_impl/windows/util.rs +++ b/src/platform_impl/windows/util.rs @@ -76,7 +76,7 @@ pub fn get_client_rect(hwnd: HWND) -> Result { } } -pub fn adjust_size(hwnd: HWND, size: PhysicalSize) -> PhysicalSize { +pub fn adjust_size(hwnd: HWND, size: PhysicalSize) -> PhysicalSize { let (width, height): (u32, u32) = size.into(); let rect = RECT { left: 0, diff --git a/src/platform_impl/windows/window.rs b/src/platform_impl/windows/window.rs index 481ccdccb8..8a89c44ecf 100644 --- a/src/platform_impl/windows/window.rs +++ b/src/platform_impl/windows/window.rs @@ -147,24 +147,24 @@ impl Window { } #[inline] - pub fn outer_position(&self) -> Result { + pub fn outer_position(&self) -> Result, NotSupportedError> { util::get_window_rect(self.window.0) - .map(|rect| Ok(PhysicalPosition::new(rect.left as f64, rect.top as f64))) + .map(|rect| Ok(PhysicalPosition::new(rect.left as i32, rect.top as i32))) .expect("Unexpected GetWindowRect failure; please report this error to https://github.com/rust-windowing/winit") } #[inline] - pub fn inner_position(&self) -> Result { + pub fn inner_position(&self) -> Result, NotSupportedError> { let mut position: POINT = unsafe { mem::zeroed() }; if unsafe { winuser::ClientToScreen(self.window.0, &mut position) } == 0 { panic!("Unexpected ClientToScreen failure: please report this error to https://github.com/rust-windowing/winit") } - Ok(PhysicalPosition::new(position.x as f64, position.y as f64)) + Ok(PhysicalPosition::new(position.x as i32, position.y as i32)) } #[inline] pub fn set_outer_position(&self, position: Position) { - let (x, y): (i32, i32) = position.to_physical(self.hidpi_factor()).into(); + let (x, y): (i32, i32) = position.to_physical::(self.hidpi_factor()).into(); let window_state = Arc::clone(&self.window_state); let window = self.window.clone(); @@ -192,7 +192,7 @@ impl Window { } #[inline] - pub fn inner_size(&self) -> PhysicalSize { + pub fn inner_size(&self) -> PhysicalSize { let mut rect: RECT = unsafe { mem::zeroed() }; if unsafe { winuser::GetClientRect(self.window.0, &mut rect) } == 0 { panic!("Unexpected GetClientRect failure: please report this error to https://github.com/rust-windowing/winit") @@ -204,7 +204,7 @@ impl Window { } #[inline] - pub fn outer_size(&self) -> PhysicalSize { + pub fn outer_size(&self) -> PhysicalSize { util::get_window_rect(self.window.0) .map(|rect| { PhysicalSize::new( @@ -250,7 +250,7 @@ impl Window { #[inline] pub fn set_inner_size(&self, size: Size) { let dpi_factor = self.hidpi_factor(); - let (width, height) = size.to_physical(dpi_factor).into(); + let (width, height) = size.to_physical::(dpi_factor).into(); let window_state = Arc::clone(&self.window_state); let window = self.window.clone(); @@ -363,7 +363,7 @@ impl Window { #[inline] pub fn set_cursor_position(&self, position: Position) -> Result<(), ExternalError> { let dpi_factor = self.hidpi_factor(); - let (x, y) = position.to_physical(dpi_factor).into(); + let (x, y) = position.to_physical::(dpi_factor).into(); let mut point = POINT { x, y }; unsafe { diff --git a/src/window.rs b/src/window.rs index b97b9c6a58..b961f41030 100644 --- a/src/window.rs +++ b/src/window.rs @@ -422,7 +422,7 @@ impl Window { /// /// [safe area]: https://developer.apple.com/documentation/uikit/uiview/2891103-safeareainsets?language=objc #[inline] - pub fn inner_position(&self) -> Result { + pub fn inner_position(&self) -> Result, NotSupportedError> { self.window.inner_position() } @@ -441,7 +441,7 @@ impl Window { /// - **iOS:** Can only be called on the main thread. Returns the top left coordinates of the /// window in the screen space coordinate system. #[inline] - pub fn outer_position(&self) -> Result { + pub fn outer_position(&self) -> Result, NotSupportedError> { self.window.outer_position() } @@ -470,7 +470,7 @@ impl Window { /// /// [safe area]: https://developer.apple.com/documentation/uikit/uiview/2891103-safeareainsets?language=objc #[inline] - pub fn inner_size(&self) -> PhysicalSize { + pub fn inner_size(&self) -> PhysicalSize { self.window.inner_size() } @@ -498,7 +498,7 @@ impl Window { /// - **iOS:** Can only be called on the main thread. Returns the `PhysicalSize` of the window in /// screen space coordinates. #[inline] - pub fn outer_size(&self) -> PhysicalSize { + pub fn outer_size(&self) -> PhysicalSize { self.window.outer_size() } diff --git a/tests/serde_objects.rs b/tests/serde_objects.rs index df51898861..ad729dcd1b 100644 --- a/tests/serde_objects.rs +++ b/tests/serde_objects.rs @@ -31,8 +31,9 @@ fn events_serde() { #[test] fn dpi_serde() { - needs_serde::(); - needs_serde::(); - needs_serde::(); - needs_serde::(); + needs_serde::>(); + needs_serde::>(); + needs_serde::>(); + needs_serde::>(); + needs_serde::>(); }