changeset 3936:c5c3c772f5aa SDL-1.2

Let app set SDL_VIDEO_ALLOW_SCREENSAVER environment variable to override SDL's attempt to disable screen savers. Works for Quartz (Mac OS X) and X11. Need a formal API for this in 1.3, still. Fixes Bugzilla #415.
author Ryan C. Gordon <icculus@icculus.org>
date Fri, 06 Apr 2007 20:30:41 +0000
parents 00959c391aa6
children 5c6e937518c6
files src/video/quartz/SDL_QuartzEvents.m src/video/quartz/SDL_QuartzVideo.h src/video/quartz/SDL_QuartzVideo.m src/video/x11/SDL_x11events.c src/video/x11/SDL_x11events_c.h src/video/x11/SDL_x11video.c src/video/x11/SDL_x11video.h
diffstat 7 files changed, 37 insertions(+), 11 deletions(-) [+]
line wrap: on
line diff
--- a/src/video/quartz/SDL_QuartzEvents.m	Wed Apr 04 10:34:42 2007 +0000
+++ b/src/video/quartz/SDL_QuartzEvents.m	Fri Apr 06 20:30:41 2007 +0000
@@ -734,11 +734,13 @@
         return;  /* don't do anything if there's no screen surface. */
 
     /* Update activity every five seconds to prevent screensaver. --ryan. */
-    nowTicks = SDL_GetTicks();
-    if ((nowTicks - screensaverTicks) > 5000)
-    {
-        UpdateSystemActivity(UsrActivity);
-        screensaverTicks = nowTicks;
+    if (!allow_screensaver) {
+        nowTicks = SDL_GetTicks();
+        if ((nowTicks - screensaverTicks) > 5000)
+        {
+            UpdateSystemActivity(UsrActivity);
+            screensaverTicks = nowTicks;
+        }
     }
 
     pool = [ [ NSAutoreleasePool alloc ] init ];
--- a/src/video/quartz/SDL_QuartzVideo.h	Wed Apr 04 10:34:42 2007 +0000
+++ b/src/video/quartz/SDL_QuartzVideo.h	Fri Apr 06 20:30:41 2007 +0000
@@ -80,6 +80,7 @@
 /* Main driver structure to store required state information */
 typedef struct SDL_PrivateVideoData {
 
+    BOOL               allow_screensaver;  /* 0 == disable screensaver */
     CGDirectDisplayID  display;            /* 0 == main display (only support single display) */
     CFDictionaryRef    mode;               /* current mode of the display */
     CFDictionaryRef    save_mode;          /* original mode of the display */
@@ -127,6 +128,7 @@
 #define display_id (this->hidden->display)
 #define mode (this->hidden->mode)
 #define save_mode (this->hidden->save_mode)
+#define allow_screensaver (this->hidden->allow_screensaver)
 #define mode_list (this->hidden->mode_list)
 #define palette (this->hidden->palette)
 #define gl_context (this->hidden->gl_context)
--- a/src/video/quartz/SDL_QuartzVideo.m	Wed Apr 04 10:34:42 2007 +0000
+++ b/src/video/quartz/SDL_QuartzVideo.m	Fri Apr 06 20:30:41 2007 +0000
@@ -169,12 +169,17 @@
 
 static int QZ_VideoInit (_THIS, SDL_PixelFormat *video_format) {
 
+    const char *env = NULL;
+
     /* Initialize the video settings; this data persists between mode switches */
     display_id = kCGDirectMainDisplay;
     save_mode  = CGDisplayCurrentMode    (display_id);
     mode_list  = CGDisplayAvailableModes (display_id);
     palette    = CGPaletteCreateDefaultColorPalette ();
 
+    env = SDL_getenv("SDL_VIDEO_ALLOW_SCREENSAVER");
+    allow_screensaver = ( env && SDL_atoi(env) ) ? YES : NO;
+
     /* Gather some information that is useful to know about the display */
     CFNumberGetValue (CFDictionaryGetValue (save_mode, kCGDisplayBitsPerPixel),
                       kCFNumberSInt32Type, &device_bpp);
--- a/src/video/x11/SDL_x11events.c	Wed Apr 04 10:34:42 2007 +0000
+++ b/src/video/x11/SDL_x11events.c	Fri Apr 06 20:30:41 2007 +0000
@@ -1136,9 +1136,14 @@
 #endif /* SDL_VIDEO_DRIVER_X11_DPMS */
 }
 
-void X11_DisableScreenSaver(Display *display)
+void X11_DisableScreenSaver(_THIS, Display *display)
 {
 	int timeout, interval, prefer_blank, allow_exp;
+
+	if (this->hidden->allow_screensaver) {
+		return;
+	}
+
 	XGetScreenSaver(display, &timeout, &interval, &prefer_blank, &allow_exp);
 	timeout = 0;
 	XSetScreenSaver(display, timeout, interval, prefer_blank, allow_exp);
@@ -1153,9 +1158,14 @@
 #endif /* SDL_VIDEO_DRIVER_X11_DPMS */
 }
 
-void X11_RestoreScreenSaver(Display *display, int saved_timeout, BOOL dpms)
+void X11_RestoreScreenSaver(_THIS, Display *display, int saved_timeout, BOOL dpms)
 {
 	int timeout, interval, prefer_blank, allow_exp;
+
+	if (this->hidden->allow_screensaver) {
+		return;
+	}
+
 	XGetScreenSaver(display, &timeout, &interval, &prefer_blank, &allow_exp);
 	timeout = saved_timeout;
 	XSetScreenSaver(display, timeout, interval, prefer_blank, allow_exp);
--- a/src/video/x11/SDL_x11events_c.h	Wed Apr 04 10:34:42 2007 +0000
+++ b/src/video/x11/SDL_x11events_c.h	Fri Apr 06 20:30:41 2007 +0000
@@ -29,5 +29,5 @@
 extern void X11_SetKeyboardState(Display *display, const char *key_vec);
 
 extern void X11_SaveScreenSaver(Display *display, int *saved_timeout, BOOL *dpms);
-extern void X11_DisableScreenSaver(Display *display);
-extern void X11_RestoreScreenSaver(Display *display, int saved_timeout, BOOL dpms);
+extern void X11_DisableScreenSaver(_THIS, Display *display);
+extern void X11_RestoreScreenSaver(_THIS, Display *display, int saved_timeout, BOOL dpms);
--- a/src/video/x11/SDL_x11video.c	Wed Apr 04 10:34:42 2007 +0000
+++ b/src/video/x11/SDL_x11video.c	Fri Apr 06 20:30:41 2007 +0000
@@ -441,6 +441,7 @@
 
 static int X11_VideoInit(_THIS, SDL_PixelFormat *vformat)
 {
+	const char *env = NULL;
 	char *display;
 	int i;
 
@@ -546,7 +547,7 @@
 
 	/* Save DPMS and screensaver settings */
 	X11_SaveScreenSaver(SDL_Display, &screensaver_timeout, &dpms_enabled);
-	X11_DisableScreenSaver(SDL_Display);
+	X11_DisableScreenSaver(this, SDL_Display);
 
 	/* See if we have been passed a window to use */
 	SDL_windowid = SDL_getenv("SDL_WINDOWID");
@@ -562,6 +563,10 @@
 	/* Fill in some window manager capabilities */
 	this->info.wm_available = 1;
 
+	/* Allow environment override of screensaver disable. */
+	env = SDL_getenv("SDL_VIDEO_ALLOW_SCREENSAVER");
+	this->hidden->allow_screensaver = ( (env && SDL_atoi(env)) ? 1 : 0 );
+
 	/* We're done! */
 	XFlush(SDL_Display);
 	return(0);
@@ -1375,7 +1380,7 @@
 		}
 
 		/* Restore DPMS and screensaver settings */
-		X11_RestoreScreenSaver(SDL_Display, screensaver_timeout, dpms_enabled);
+		X11_RestoreScreenSaver(this, SDL_Display, screensaver_timeout, dpms_enabled);
 
 		/* Free that blank cursor */
 		if ( SDL_BlankCursor != NULL ) {
--- a/src/video/x11/SDL_x11video.h	Wed Apr 04 10:34:42 2007 +0000
+++ b/src/video/x11/SDL_x11video.h	Fri Apr 06 20:30:41 2007 +0000
@@ -139,6 +139,8 @@
     int use_xme;
     int currently_fullscreen;
 
+    int allow_screensaver;
+
     /* Automatic mode switching support (entering/leaving fullscreen) */
     Uint32 switch_waiting;
     Uint32 switch_time;