[PATCH] Update lastDeviceEventTime in XResetScreenSaver

Keith Packard keithp at keithp.com
Sat Jan 2 10:45:49 PST 2010


On Sat,  2 Jan 2010 11:58:40 +0000, Julien Cristau <jcristau at debian.org> wrote:

> --- a/dix/dispatch.c
> +++ b/dix/dispatch.c
> @@ -3517,6 +3517,11 @@ int ProcForceScreenSaver(ClientPtr client)
>      rc = dixSaveScreens(client, SCREEN_SAVER_FORCER, (int)stuff->mode);
>      if (rc != Success)
>  	return rc;
> +    if (stuff->mode == ScreenSaverReset)
> +    {
> +        UpdateCurrentTimeIf();
> +        lastDeviceEventTime = currentTime;
> +    }

Seems like it would be better to stick this back in dixSaveScreens where
it used to be, but restricted to only being done in the
SCREE_SAVER_FORCER path when mode is ScreenSaverReset. That way you pick
up all of the other ScreenSaverReset cases as well (VT switch,
suspend/resume and server startup/shutdown).

-- 
keith.packard at intel.com
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
Url : http://lists.x.org/archives/xorg-devel/attachments/20100102/2fc1bc5e/attachment.pgp 


More information about the xorg-devel mailing list