On Sun, Mar 11, 2012 at 8:05 PM, David Herrmann <[email protected]> wrote: > Udev provides a convenient helper. Use it instead of working with the > property-list directly. > > Signed-off-by: David Herrmann <[email protected]> > --- > src/compositor-drm.c | 10 ++++------ > 1 file changed, 4 insertions(+), 6 deletions(-) > > diff --git a/src/compositor-drm.c b/src/compositor-drm.c > index 27d4512..ba376eb 100644 > --- a/src/compositor-drm.c > +++ b/src/compositor-drm.c > @@ -1396,20 +1396,18 @@ update_outputs(struct drm_compositor *ec, struct > udev_device *drm_device) > static int > udev_event_is_hotplug(struct drm_compositor *ec, struct udev_device *device) > { > - struct udev_list_entry *list, *hotplug_entry; > const char *sysnum; > + const char *val; > > sysnum = udev_device_get_sysnum(device); > if (!sysnum || atoi(sysnum) != ec->drm.id) > return 0; > > - list = udev_device_get_properties_list_entry(device); > - > - hotplug_entry = udev_list_entry_get_by_name(list, "HOTPLUG"); > - if (hotplug_entry == NULL) > + val = udev_device_get_property_value(device, "HOTPLUG"); > + if (!val) > return 0; > > - return strcmp(udev_list_entry_get_value(hotplug_entry), "1") == 0; > + return strcmp(val, "1") == 0; > } > > static int
I forgot to mention that it depends on my previous patch: [PATCH] compositor-drm: ignore hotplug-events from other devices The conflict, if the previous patch is not applied, should be trivial, though. Regards David _______________________________________________ wayland-devel mailing list [email protected] http://lists.freedesktop.org/mailman/listinfo/wayland-devel
