From 4b5d16554b614069a89672dd282cf4eaeccb0d40 Mon Sep 17 00:00:00 2001
From: Robin Gareus <robin@gareus.org>
Date: Sun, 11 Aug 2019 19:04:02 +0200
Subject: Simplify screensaver inhibit API and fix OSX compatibility

---
 gtk2_ardour/screensaver.cc | 57 +++++++++++++++++++++++-----------------------
 gtk2_ardour/utils.h        |  3 +--
 2 files changed, 29 insertions(+), 31 deletions(-)

diff --git a/gtk2_ardour/screensaver.cc b/gtk2_ardour/screensaver.cc
index a549a5710a..e4aab918f4 100644
--- a/gtk2_ardour/screensaver.cc
+++ b/gtk2_ardour/screensaver.cc
@@ -23,15 +23,13 @@
 #include <windows.h>
 
 void
-ARDOUR_UI_UTILS::enable_screensaver ()
+ARDOUR_UI_UTILS::inhibit_screensaver (bool inhibit)
 {
-	SetThreadExecutionState (ES_DISPLAY_REQUIRED | ES_SYSTEM_REQUIRED | ES_CONTINUOUS);
-}
-
-void
-ARDOUR_UI_UTILS::disable_screensaver ()
-{
-	SetThreadExecutionState (ES_CONTINUOUS);
+	if (inhibit) {
+		SetThreadExecutionState (ES_DISPLAY_REQUIRED | ES_SYSTEM_REQUIRED | ES_CONTINUOUS);
+	} else {
+		SetThreadExecutionState (ES_CONTINUOUS);
+	}
 }
 
 #elif defined __APPLE__
@@ -42,23 +40,26 @@ static IOReturn success = kIOReturnError;
 static IOPMAssertionID assertion_id;
 
 void
-ARDOUR_UI_UTILS::enable_screensaver ()
-{
-	static const CFStringRef name = CFSTR("Ardour DAW");
-	/* kIOPMAssertionTypeNoDisplaySleep prevents display sleep,
-	 * kIOPMAssertionTypeNoIdleSleep prevents idle sleep
-	 */
-	success = IOPMAssertionCreateWithName (kIOPMAssertionTypeNoDisplaySleep, kIOPMAssertionLevelOn, name, &assertion_id);
-}
-
-void
-ARDOUR_UI_UTILS::disable_screensaver ()
+ARDOUR_UI_UTILS::inhibit_screensaver (bool inhibit)
 {
-	if (success != kIOReturnSuccess) {
+	if (inhibit == (success == kIOReturnSuccess)) {
 		return;
 	}
-	if (kIOReturnSuccess == IOPMAssertionRelease (assertion_id)) {
-		success = kIOReturnError;
+
+	if (inhibit) {
+		/* kIOPMAssertionTypeNoDisplaySleep prevents display sleep,
+		 * kIOPMAssertionTypeNoIdleSleep prevents idle sleep
+		 */
+#ifdef __ppc__ /* OS X 10.5 compat API */
+		success = IOPMAssertionCreate (kIOPMAssertionTypeNoDisplaySleep, kIOPMAssertionLevelOn, &assertion_id);
+#else
+		static const CFStringRef name = CFSTR("Ardour DAW");
+		success = IOPMAssertionCreateWithName (kIOPMAssertionTypeNoDisplaySleep, kIOPMAssertionLevelOn, name, &assertion_id);
+#endif
+	} else {
+		if (kIOReturnSuccess == IOPMAssertionRelease (assertion_id)) {
+			success = kIOReturnError; // mark as inactive
+		}
 	}
 }
 
@@ -88,15 +89,13 @@ xdg_screensaver_reset ()
 }
 
 void
-ARDOUR_UI_UTILS::enable_screensaver ()
-{
-	glib_timer = Glib::signal_timeout().connect_seconds (sigc::ptr_fun (&xdg_screensaver_reset), 45, Glib::PRIORITY_DEFAULT_IDLE);
-}
-
-void
-ARDOUR_UI_UTILS::disable_screensaver ()
+ARDOUR_UI_UTILS::inhibit_screensaver (bool inhibit)
 {
 	glib_timer.disconnect ();
+	if (inhibit) {
+		xdg_screensaver_reset ();
+		glib_timer = Glib::signal_timeout().connect_seconds (sigc::ptr_fun (&xdg_screensaver_reset), 45, Glib::PRIORITY_DEFAULT_IDLE);
+	}
 }
 
 #endif
diff --git a/gtk2_ardour/utils.h b/gtk2_ardour/utils.h
index d3e8e11fdc..dc7b975cc1 100644
--- a/gtk2_ardour/utils.h
+++ b/gtk2_ardour/utils.h
@@ -107,8 +107,7 @@ bool windows_overlap (Gtk::Window *a, Gtk::Window *b);
 bool overwrite_file_dialog (Gtk::Window& parent, std::string title, std::string text);
 bool running_from_source_tree ();
 
-void enable_screensaver ();
-void disable_screensaver ();
+void inhibit_screensaver (bool);
 
 } // namespace
 #endif /* __ardour_gtk_utils_h__ */
-- 
cgit v1.2.3