]> git.alsa-project.org Git - alsa-utils.git/commitdiff
alsactl: remove open-coded search
authorClemens Ladisch <clemens@ladisch.de>
Wed, 26 May 2010 08:19:17 +0000 (10:19 +0200)
committerClemens Ladisch <clemens@ladisch.de>
Wed, 26 May 2010 08:19:17 +0000 (10:19 +0200)
Remove search_comment_item since it does the same as snd_config_search.

Signed-off-by: Clemens Ladisch <clemens@ladisch.de>
alsactl/state.c

index 86f77482477cad02548f6cf8cbaeaad4d45567e3..815345af14fc621bd02babe33353bb039ed4a957 100644 (file)
@@ -899,32 +899,15 @@ static int add_user_control(snd_ctl_t *handle, snd_ctl_elem_info_t *info, snd_co
        return snd_ctl_elem_info(handle, info);
 }
 
-/*
- * look for a config node with the given item name
- */
-static snd_config_t *search_comment_item(snd_config_t *conf, const char *name)
-{
-       snd_config_iterator_t i, next;
-       snd_config_for_each(i, next, conf) {
-               snd_config_t *n = snd_config_iterator_entry(i);
-               const char *id;
-               if (snd_config_get_id(n, &id) < 0)
-                       continue;
-               if (strcmp(id, name) == 0)
-                       return n;
-       }
-       return NULL;
-}
-
 /*
  * check whether the config item has the same of compatible type
  */
 static int check_comment_type(snd_config_t *conf, int type)
 {
-       snd_config_t *n = search_comment_item(conf, "type");
+       snd_config_t *n;
        int ctype;
 
-       if (!n)
+       if (snd_config_search(conf, "type", &n) < 0)
                return 0; /* not defined */
        ctype = get_comment_type(n);
        if (ctype == type)
@@ -980,8 +963,7 @@ static int check_comment_range(snd_ctl_t *handle, snd_config_t *conf,
        long ndbmin, ndbmax;
        snd_ctl_elem_id_t *id;
 
-       n = search_comment_item(conf, "range");
-       if (!n)
+       if (snd_config_search(conf, "range", &n) < 0)
                return 0;
        if (get_comment_range(n, SND_CTL_ELEM_TYPE_INTEGER,
                              &omin, &omax, &ostep) < 0)
@@ -996,13 +978,11 @@ static int check_comment_range(snd_ctl_t *handle, snd_config_t *conf,
        if (omin >= omax || nmin >= nmax)
                return 0; /* invalid values */
 
-       n = search_comment_item(conf, "dbmin");
-       if (!n)
+       if (snd_config_search(conf, "dbmin", &n) < 0)
                return 0;
        if (config_integer(n, &odbmin, doit) < 0)
                return 0;
-       n = search_comment_item(conf, "dbmax");
-       if (!n)
+       if (snd_config_search(conf, "dbmax", &n) < 0)
                return 0;
        if (config_integer(n, &odbmax, doit) < 0)
                return 0;