]> git.alsa-project.org Git - alsa-lib.git/commitdiff
topology: apply the same type to compared two variables
authorTakashi Sakamoto <o-takashi@sakamocchi.jp>
Mon, 11 Jul 2016 12:05:59 +0000 (21:05 +0900)
committerTakashi Iwai <tiwai@suse.de>
Mon, 11 Jul 2016 13:24:12 +0000 (15:24 +0200)
This commit fixes below compiler warnings.

data.c: In function ‘get_token_value’:
data.c:262:16: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
  for (i = 0; i < tokens->num_tokens; i++) {
                ^
data.c: In function ‘copy_tuples’:
data.c:351:16: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
  for (i = 0; i < tuples->num_sets ; i++) {
                ^
data.c:378:17: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
   for (j = 0; j < tuple_set->num_tuples; j++) {
                 ^
data.c: In function ‘tplg_free_tuples’:
data.c:723:16: warning: comparison between signed and unsigned integer expressions [-Wsign-compare]
  for (i = 0; i < tuples->num_sets; i++)
                ^

Signed-off-by: Takashi Sakamoto <o-takashi@sakamocchi.jp>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
src/topology/data.c

index 4ecd2c2ff76c791f1fff8c667df9610b45d061e5..93eaf9192348f1a106cf76dfc9917ab63af2f4aa 100644 (file)
@@ -257,7 +257,7 @@ static int tplg_parse_data_hex(snd_config_t *cfg, struct tplg_elem *elem,
 static int get_token_value(const char *token_id,
        struct tplg_vendor_tokens *tokens)
 {
-       int i;
+       unsigned int i;
 
        for (i = 0; i < tokens->num_tokens; i++) {
                if (strcmp(token_id, tokens->token[i].id) == 0)
@@ -338,7 +338,8 @@ static int copy_tuples(struct tplg_elem *elem,
        struct snd_soc_tplg_vendor_string_elem *string;
        struct snd_soc_tplg_vendor_value_elem *value;
        int set_size, size, off;
-       int i, j, token_val;
+       unsigned int i, j;
+       int token_val;
 
        if (priv) {
                SNDERR("error: %s has more data than tuples\n", elem->id);
@@ -713,7 +714,7 @@ int tplg_parse_tuples(snd_tplg_t *tplg, snd_config_t *cfg,
 void tplg_free_tuples(void *obj)
 {
        struct tplg_vendor_tuples *tuples = (struct tplg_vendor_tuples *)obj;
-       int i;
+       unsigned int i;
 
        if (!tuples || !tuples->set)
                return;