[PATCH v2 6/9] Xext: store the bracket values for idle counters in the private

[Peter Hutterer]

On Thu, Mar 15, 2012 at 08:12:20AM -0700, Jamey Sharp wrote:
> A small detail:
> 
> On 3/14/12, Peter Hutterer <peter.hutterer at who-t.net> wrote:
> >  static void
> >  IdleTimeWakeupHandler (pointer pCounter, int rc, pointer LastSelectMask)
> >  {
> > -    SyncCounter *counter = IdleTimeCounter;
> > +    SyncCounter *counter = pCounter;
> >...
> > @@ -2814,7 +2817,7 @@ IdleTimeWakeupHandler (pointer pCounter, int rc,
> > pointer LastSelectMask)
> >      if ((greater && XSyncValueGreaterOrEqual (idle, *greater)) ||
> >          (less && XSyncValueLessOrEqual (idle, *less)))
> >      {
> > -	SyncChangeCounter (counter, idle);
> > +	SyncChangeCounter ((SyncCounter*)pCounter, idle);
> >      }
> >  }
> 
> You have a properly-typed "counter" in scope now, so you can drop this hunk.

fixed, thanks.
 
Cheers,
  Peter