Ok, I failed to actually notice the patch was already on the list in my deluge 
of unread messages.  Seeing as how this is exactly what I just described in my 
previous email,

Reviewed-by: Jeremy Huddleston <[email protected]>

Thanks,
Jeremy

On Jun 14, 2012, at 9:15 AM, Andy Ritger <[email protected]> wrote:

> Call UpdateCurrentTimeIf(), not UpdateCurrentTime(), from RRTellChanged().
> The latter calls ProcessInputEvents(), which can trigger a recursion
> into mieqProcessInputEvents().  The former omits the call to
> ProcessInputEvents().
> 
> Signed-off-by: Andy Ritger <[email protected]>
> ---
> randr/randr.c |    2 +-
> 1 files changed, 1 insertions(+), 1 deletions(-)
> 
> diff --git a/randr/randr.c b/randr/randr.c
> index a64aae3..4d4298a 100644
> --- a/randr/randr.c
> +++ b/randr/randr.c
> @@ -416,7 +416,7 @@ RRTellChanged(ScreenPtr pScreen)
>     int i;
> 
>     if (pScrPriv->changed) {
> -        UpdateCurrentTime();
> +        UpdateCurrentTimeIf();
>         if (pScrPriv->configChanged) {
>             pScrPriv->lastConfigTime = currentTime;
>             pScrPriv->configChanged = FALSE;
> -- 
> 1.7.7
> 
> _______________________________________________
> [email protected]: X.Org development
> Archives: http://lists.x.org/archives/xorg-devel
> Info: http://lists.x.org/mailman/listinfo/xorg-devel
> 

_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel

Reply via email to