Change Suspended(true/false) => Suspended/Resumed (#959)

CHANGELOG.md

@@ -1,5 +1,7 @@
 # Unreleased
 
+- Change `Event::Suspended(true / false)` to `Event::Suspended` and `Event::Resumed`.
+
 # 0.20.0 Alpha 1
 
 - Changes below are considered **breaking**.

src/event.rs

@@ -37,10 +37,11 @@ pub enum Event<T> {
     /// emitted, it is guaranteed to be the last event emitted.
     LoopDestroyed,
 
-    /// Emitted when the application has been suspended or resumed.
+    /// Emitted when the application has been suspended.
-    ///
+    Suspended,
-    /// The parameter is true if app was suspended, and false if it has been resumed.
+
-    Suspended(bool),
+    /// Emitted when the application has been resumed.
+    Resumed,
 }
 
 impl<T> Event<T> {
@@ -53,7 +54,8 @@ impl<T> Event<T> {
             NewEvents(cause) => Ok(NewEvents(cause)),
             EventsCleared => Ok(EventsCleared),
             LoopDestroyed => Ok(LoopDestroyed),
-            Suspended(suspended) => Ok(Suspended(suspended)),
+            Suspended => Ok(Suspended),
+            Resumed => Ok(Resumed),
         }
     }
 }

src/platform_impl/android/mod.rs

@@ -85,14 +85,14 @@ impl EventLoop {
                     if let Some(cb) = self.suspend_callback.borrow().as_ref() {
                         (*cb)(false);
                     }
-                    Some(Event::Suspended(false))
+                    Some(Event::Resumed)
                 },
                 android_glue::Event::TermWindow => {
                     // The activity went to background.
                     if let Some(cb) = self.suspend_callback.borrow().as_ref() {
                         (*cb)(true);
                     }
-                    Some(Event::Suspended(true))
+                    Some(Event::Suspended)
                 },
                 android_glue::Event::WindowResized | android_glue::Event::ConfigChanged => {
                     // Activity Orientation changed or resized.

src/platform_impl/ios/mod.rs

@@ -46,8 +46,8 @@
 //!
 //! This is how those event are represented in winit:
 //!
-//!  - applicationDidBecomeActive is Suspended(false)
+//!  - applicationDidBecomeActive is Resumed
-//!  - applicationWillResignActive is Suspended(true)
+//!  - applicationWillResignActive is Suspended
 //!  - applicationWillTerminate is LoopDestroyed
 //!
 //! Keep in mind that after LoopDestroyed event is received every attempt to draw with

src/platform_impl/ios/view.rs

@@ -382,11 +382,11 @@ pub fn create_delegate_class() {
     }
 
     extern "C" fn did_become_active(_: &Object, _: Sel, _: id) {
-        unsafe { AppState::handle_nonuser_event(Event::Suspended(false)) }
+        unsafe { AppState::handle_nonuser_event(Event::Resumed) }
     }
 
     extern "C" fn will_resign_active(_: &Object, _: Sel, _: id) {
-        unsafe { AppState::handle_nonuser_event(Event::Suspended(true)) }
+        unsafe { AppState::handle_nonuser_event(Event::Suspended) }
     }
 
     extern "C" fn will_enter_foreground(_: &Object, _: Sel, _: id) {}

src/platform_impl/macos/app_delegate.rs

@@ -54,7 +54,7 @@ extern "C" fn did_finish_launching(_: &Object, _: Sel, _: id) -> BOOL {
 extern "C" fn did_become_active(_: &Object, _: Sel, _: id) {
     trace!("Triggered `didBecomeActive`");
     /*unsafe {
-        HANDLER.lock().unwrap().handle_nonuser_event(Event::Suspended(false))
+        HANDLER.lock().unwrap().handle_nonuser_event(Event::Resumed)
     }*/
     trace!("Completed `didBecomeActive`");
 }
@@ -62,7 +62,7 @@ extern "C" fn did_become_active(_: &Object, _: Sel, _: id) {
 extern "C" fn will_resign_active(_: &Object, _: Sel, _: id) {
     trace!("Triggered `willResignActive`");
     /*unsafe {
-        HANDLER.lock().unwrap().handle_nonuser_event(Event::Suspended(true))
+        HANDLER.lock().unwrap().handle_nonuser_event(Event::Suspended)
     }*/
     trace!("Completed `willResignActive`");
 }