[PATCH 1/4] xserver: process udev "changed" and "add" events in the same code paths

Lennart Poettering lennart at poettering.net
Wed Jul 20 11:05:44 PDT 2011


On Wed, 20.07.11 14:14, Peter Hutterer (peter.hutterer at who-t.net) wrote:

> > --- a/config/udev.c
> > +++ b/config/udev.c
> > @@ -251,14 +251,12 @@ wakeup_handler(pointer data, int err, pointer read_mask)
> >              return;
> >          action = udev_device_get_action(udev_device);
> >          if (action) {
> > -            if (!strcmp(action, "add"))
> > -                device_added(udev_device);
> > -            else if (!strcmp(action, "remove"))
> > -                device_removed(udev_device);
> > -            else if (!strcmp(action, "change")) {
> > +            if (!strcmp(action, "add") || !strcmp(action, "change")) {
> >                  device_removed(udev_device);
> >                  device_added(udev_device);
> >              }
> > +            else if (!strcmp(action, "remove"))
> > +                device_removed(udev_device);
> >          }
> >          udev_device_unref(udev_device);
> >      }
> 
> merged all four, thanks.
> 
> can I assume your Signed-off-by?
> (I guess I don't have to since we're working for the same corp, but...)

Yes, of course. Wasn't aware you guys were using S-o-b. Otherwise I'd
have included it.

Thanks for merging!

Lennart

-- 
Lennart Poettering - Red Hat, Inc.


More information about the xorg-devel mailing list