Set cursor only when its visibility changes

3 files changed, 11 insertions, 7 deletions

diff --git a/src/display.rs b/src/display.rs
index bdb5274c..3d13f465 100644
--- a/src/display.rs
+++ b/src/display.rs
@@ -231,8 +231,8 @@ impl Display {
         self.tx.clone()
     }
 
-    pub fn window(&self) -> &Window {
-        &self.window
+    pub fn window(&mut self) -> &mut Window {
+        &mut self.window
     }
 
     /// Process pending resize events
diff --git a/src/event.rs b/src/event.rs
index d6b0796a..fd33bfb3 100644
--- a/src/event.rs
+++ b/src/event.rs
@@ -261,7 +261,7 @@ impl<N: Notify> Processor<N> {
     pub fn process_events<'a>(
         &mut self,
         term: &'a FairMutex<Term>,
-        window: &Window
+        window: &mut Window
     ) -> MutexGuard<'a, Term> {
         // Terminal is lazily initialized the first time an event is returned
         // from the blocking WaitEventsIterator. Otherwise, the pty reader would
diff --git a/src/window.rs b/src/window.rs
index b90c242d..fa751352 100644
--- a/src/window.rs
+++ b/src/window.rs
@@ -53,6 +53,7 @@ type Result<T> = ::std::result::Result<T, Error>;
 /// Wraps the underlying windowing library to provide a stable API in Alacritty
 pub struct Window {
     glutin_window: glutin::Window,
+    cursor_visible: bool,
 }
 
 /// Threadsafe APIs for the window
@@ -218,6 +219,7 @@ impl Window {
 
         Ok(Window {
             glutin_window: window,
+            cursor_visible: true,
         })
     }
 
@@ -290,10 +292,12 @@ impl Window {
     }
 
     /// Set cursor visible
-    #[inline]
-    pub fn set_cursor_visible(&self, show: bool) {
-        self.glutin_window.set_cursor(if show { glutin::MouseCursor::Default }
-                                      else { glutin::MouseCursor::NoneCursor });
+    pub fn set_cursor_visible(&mut self, visible: bool) {
+        if visible != self.cursor_visible {
+            self.cursor_visible = visible;
+            self.glutin_window.set_cursor(if visible { glutin::MouseCursor::Default }
+                                          else { glutin::MouseCursor::NoneCursor });
+        }
     }
 }