From 309c2a2ce95aae54879b4957d113f03608530c15 Mon Sep 17 00:00:00 2001 From: Lennart Poettering Date: Wed, 13 Jul 2011 20:11:23 +0200 Subject: loginctl: use new udev_device_has_tag() call from libudev where applicable --- src/logind-dbus.c | 19 +------------------ src/logind.h | 2 -- src/sysfs-show.c | 21 ++------------------- 3 files changed, 3 insertions(+), 39 deletions(-) (limited to 'src') diff --git a/src/logind-dbus.c b/src/logind-dbus.c index 16dbd3664c..e0855f4cf8 100644 --- a/src/logind-dbus.c +++ b/src/logind-dbus.c @@ -550,23 +550,6 @@ fail: return r; } -static bool device_has_tag(struct udev_device *d, const char *tag) { - struct udev_list_entry *first, *item; - - assert(d); - assert(tag); - - /* FIXME */ - udev_device_get_is_initialized(d); - - first = udev_device_get_tags_list_entry(d); - udev_list_entry_foreach(item, first) - if (streq(udev_list_entry_get_name(item), tag)) - return true; - - return false; -} - static int trigger_device(Manager *m, const char *prefix) { struct udev_enumerate *e; struct udev_list_entry *first, *item; @@ -628,7 +611,7 @@ static int attach_device(Manager *m, const char *seat, const char *sysfs) { if (!d) return -ENODEV; - if (!device_has_tag(d, "seat")) { + if (!udev_device_has_tag(d, "seat")) { r = -ENODEV; goto finish; } diff --git a/src/logind.h b/src/logind.h index db20bff194..cdb612805f 100644 --- a/src/logind.h +++ b/src/logind.h @@ -38,8 +38,6 @@ * direct client API * * udev: - * drop redundant udev_device_get_is_initialized() use as soon as libudev is fixed - * use device_has_tag() as soon as it is available * enumerate recursively with libudev when triggering * use sysfs in device hash table, not sysname, when fb driver is fixed * diff --git a/src/sysfs-show.c b/src/sysfs-show.c index b456bf1271..ab866a4707 100644 --- a/src/sysfs-show.c +++ b/src/sysfs-show.c @@ -26,23 +26,6 @@ #include "util.h" #include "sysfs-show.h" -static bool device_has_tag(struct udev_device *d, const char *tag) { - struct udev_list_entry *first, *item; - - assert(d); - assert(tag); - - /* FIXME */ - udev_device_get_is_initialized(d); - - first = udev_device_get_tags_list_entry(d); - udev_list_entry_foreach(item, first) - if (streq(udev_list_entry_get_name(item), tag)) - return true; - - return false; -} - static int show_sysfs_one( struct udev *udev, const char *seat, @@ -77,7 +60,7 @@ static int show_sysfs_one( sn = "seat0"; /* fixme, also check for tag 'seat' here */ - if (!streq(seat, sn) || !device_has_tag(d, "seat")) { + if (!streq(seat, sn) || !udev_device_has_tag(d, "seat")) { udev_device_unref(d); *item = udev_list_entry_get_next(*item); continue; @@ -109,7 +92,7 @@ static int show_sysfs_one( if (isempty(lookahead_sn)) lookahead_sn = "seat0"; - found = streq(seat, lookahead_sn) && device_has_tag(lookahead_d, "seat"); + found = streq(seat, lookahead_sn) && udev_device_has_tag(lookahead_d, "seat"); udev_device_unref(lookahead_d); if (found) -- cgit v1.2.3-54-g00ecf