[MPlayer-dev-eng] [PATCH] Add support for GNOME screensaver

[Attila Kinali]

On Wed, 26 Apr 2006 15:50:17 +0200
Piotr Kaczuba <pepe at attika.ath.cx> wrote:

> +#ifdef HAVE_DBUS_GLIB
> +        gnome_screensaver_enable();
> +#endif
> +    }
>      if (kdescreensaver_was_running && stop_xscreensaver)
>      {
>          system
> @@ -1738,8 +1746,12 @@
>                              allow_exp);
>      }
>      // turning off screensaver
> -    if (stop_xscreensaver)
> +    if (stop_xscreensaver) {
>          xscreensaver_disable(mDisplay);
> +#ifdef HAVE_DBUS_GLIB
> +        gnome_screensaver_disable();
> +#endif
> +    }

Shouldn't you check whether the screensaver is actualy running
before disabling it, to prevent enabling it when it wasnt
enabled?

And how does your patch behave, when gnome is installed, but
not running ?

			Attila Kinali

-- 
心をこめて聞け心をこめて話せ