[PATCH 2/4] Replace a few BUG_WARN with BUG_RETURN_VAL

Peter Hutterer peter.hutterer at who-t.net
Tue May 15 15:43:38 PDT 2012


On Tue, May 15, 2012 at 10:49:59AM -0700, Alan Coopersmith wrote:
> On 05/15/12 03:26 AM, Peter Hutterer wrote:
> > Signed-off-by: Peter Hutterer <peter.hutterer at who-t.net>
> > ---
> >  Xi/exevents.c   |   25 ++++++++++---------------
> >  dix/getevents.c |    5 +----
> >  dix/touch.c     |   17 ++++++-----------
> >  3 files changed, 17 insertions(+), 30 deletions(-)
> > 
> > diff --git a/Xi/exevents.c b/Xi/exevents.c
> > index 3aa9030..7fbaa8f 100644
> > --- a/Xi/exevents.c
> > +++ b/Xi/exevents.c
> > @@ -948,10 +948,10 @@ UpdateDeviceState(DeviceIntPtr device, DeviceEvent *event)
> >      else if (event->type == ET_ProximityOut)
> >          device->proximity->in_proximity = FALSE;
> >      else if (event->type == ET_TouchBegin) {
> > -        BUG_WARN(!b || !v);
> > -        BUG_WARN(!t);
> > +        BUG_RETURN_VAL(!b || !v, DONT_PROCESS);
> > +        BUG_RETURN_VAL(!t, DONT_PROCESS);
> >  
> > -        if (!b || !t || !b->map[key])
> > +        if (!b->map[key])
> >              return DONT_PROCESS;
> 
> It's intentional that you're now also returning DONT_PROCESS for !v where you
> weren't before, right?   (In both this hunk and the next one.)

wasn't intentional, I think that's a copy-paste error. mind you, having a
touch* event on a device without valuators is definitely a bug, so we can
leave this in. let's pretend nobody noticed ;)

Cheers,
  Peter


> 
> >          if (!(event->flags & TOUCH_POINTER_EMULATED) ||
> > @@ -963,10 +963,10 @@ UpdateDeviceState(DeviceIntPtr device, DeviceEvent *event)
> >          UpdateDeviceMotionMask(device, t->state, DeviceButtonMotionMask);
> >      }
> >      else if (event->type == ET_TouchEnd) {
> > -        BUG_WARN(!b || !v);
> > -        BUG_WARN(!t);
> > +        BUG_RETURN_VAL(!b || !v, DONT_PROCESS);
> > +        BUG_RETURN_VAL(!t, DONT_PROCESS);
> >  
> > -        if (!b || !t || t->buttonsDown <= 0 || !b->map[key])
> > +        if (t->buttonsDown <= 0 || !b->map[key])
> >              return DONT_PROCESS;
> >  
> >          if (!(event->flags & TOUCH_POINTER_EMULATED))
> 
> -- 
> 	-Alan Coopersmith-              alan.coopersmith at oracle.com
> 	 Oracle Solaris Engineering - http://blogs.oracle.com/alanc


More information about the xorg-devel mailing list