[PATCH xserver 05/12] dix: split client list retrieval out of DeliverEventToClients

Peter Hutterer peter.hutterer at who-t.net
Mon Jun 6 16:04:32 PDT 2011


On Mon, Jun 06, 2011 at 02:36:23PM +0100, Daniel Stone wrote:
> Hi,
> 
> On Thu, Jun 02, 2011 at 05:13:39PM +1000, Peter Hutterer wrote:
> > +static int
> > +GetClientsForDelivery(DeviceIntPtr dev, WindowPtr win,
> > +                      xEvent *events, Mask filter, InputClients **clients)
> >  {
> > -    int attempt;
> > -    enum EventDeliveryState rc = EVENT_SKIP;
> > -    InputClients *other;
> > +    int rc = 0;
> >  
> >      [...]
> >  
> > +    rc = 1;
> > +out:
> > +    return rc;
> > +}
> > 
> > [...]
> > 
> > +    if (!GetClientsForDelivery(dev, win, events, filter, &other))
> > +        goto out;
> 
> This would probably be better off as Bool than int.

amended locally, thanks.

Cheers,
  Peter


More information about the xorg-devel mailing list