From 535cf5b5c95ab4f26baaabb228a72eebc1cba9c5 Mon Sep 17 00:00:00 2001 From: Max Kellermann Date: Sat, 24 Jan 2009 15:27:05 +0100 Subject: [PATCH] locate: use bool instead of int Use the C99 bool type instead of integer values (1/0 or 0/-1). --- src/locate.c | 61 ++++++++++++++++++++++++++-------------------------- src/locate.h | 5 +++-- 2 files changed, 33 insertions(+), 33 deletions(-) diff --git a/src/locate.c b/src/locate.c index 501153b4b..85fe297a5 100644 --- a/src/locate.c +++ b/src/locate.c @@ -48,18 +48,18 @@ locate_parse_type(const char *str) return -1; } -static int +static bool locate_item_init(struct locate_item *item, const char *type_string, const char *needle) { item->tag = locate_parse_type(type_string); if (item->tag < 0) - return -1; + return false; item->needle = g_strdup(needle); - return 0; + return true; } struct locate_item * @@ -67,7 +67,7 @@ locate_item_new(const char *type_string, const char *needle) { struct locate_item *ret = g_new(struct locate_item, 1); - if (locate_item_init(ret, type_string, needle) < 0) { + if (!locate_item_init(ret, type_string, needle)) { free(ret); ret = NULL; } @@ -101,7 +101,7 @@ locate_item_list_parse(char *argv[], int argc, struct locate_item **arrayRet) *arrayRet = g_new(struct locate_item, argc / 2); for (i = 0, item = *arrayRet; i < argc / 2; i++, item++) { - if (locate_item_init(item, argv[i * 2], argv[i * 2 + 1]) < 0) + if (!locate_item_init(item, argv[i * 2], argv[i * 2 + 1])) goto fail; } @@ -124,13 +124,13 @@ locate_item_free(struct locate_item *item) free(item); } -static int +static bool locate_tag_search(const struct song *song, enum tag_type type, const char *str) { int i; char *duplicate; - int ret = 0; - int8_t visited_types[TAG_NUM_OF_ITEM_TYPES] = { 0 }; + bool ret = false; + bool visited_types[TAG_NUM_OF_ITEM_TYPES] = { false }; if (type == LOCATE_TAG_FILE_TYPE || type == LOCATE_TAG_ANY_TYPE) { char *uri, *p; @@ -140,17 +140,17 @@ locate_tag_search(const struct song *song, enum tag_type type, const char *str) g_free(uri); if (strstr(p, str)) - ret = 1; + ret = true; g_free(p); if (ret == 1 || type == LOCATE_TAG_FILE_TYPE) return ret; } if (!song->tag) - return 0; + return false; for (i = 0; i < song->tag->numOfItems && !ret; i++) { - visited_types[song->tag->items[i]->type] = 1; + visited_types[song->tag->items[i]->type] = true; if (type != LOCATE_TAG_ANY_TYPE && song->tag->items[i]->type != type) { continue; @@ -158,7 +158,7 @@ locate_tag_search(const struct song *song, enum tag_type type, const char *str) duplicate = g_utf8_casefold(song->tag->items[i]->value, -1); if (*str && strstr(duplicate, str)) - ret = 1; + ret = true; g_free(duplicate); } @@ -166,30 +166,30 @@ locate_tag_search(const struct song *song, enum tag_type type, const char *str) * through the song's tag, it means this field is absent from * the tag or empty. Thus, if the searched string is also * empty (first char is a \0), then it's a match as well and - * we should return 1. + * we should return true. */ if (!*str && !visited_types[type]) - return 1; + return true; return ret; } -int +bool locate_song_search(const struct song *song, int num_items, const struct locate_item *items) { for (int i = 0; i < num_items; i++) if (!locate_tag_search(song, items[i].tag, items[i].needle)) - return 0; + return false; - return 1; + return true; } -static int +static bool locate_tag_match(const struct song *song, enum tag_type type, const char *str) { int i; - int8_t visited_types[TAG_NUM_OF_ITEM_TYPES] = { 0 }; + bool visited_types[TAG_NUM_OF_ITEM_TYPES] = { false }; if (type == LOCATE_TAG_FILE_TYPE || type == LOCATE_TAG_ANY_TYPE) { char *uri = song_get_uri(song); @@ -197,46 +197,45 @@ locate_tag_match(const struct song *song, enum tag_type type, const char *str) g_free(uri); if (matches) - return 1; + return true; if (type == LOCATE_TAG_FILE_TYPE) - return 0; + return false; } if (!song->tag) - return 0; + return false; for (i = 0; i < song->tag->numOfItems; i++) { - visited_types[song->tag->items[i]->type] = 1; + visited_types[song->tag->items[i]->type] = true; if (type != LOCATE_TAG_ANY_TYPE && song->tag->items[i]->type != type) { continue; } if (0 == strcmp(str, song->tag->items[i]->value)) - return 1; + return true; } /** If the search critieron was not visited during the sweep * through the song's tag, it means this field is absent from * the tag or empty. Thus, if the searched string is also * empty (first char is a \0), then it's a match as well and - * we should return 1. + * we should return true. */ if (!*str && !visited_types[type]) - return 1; + return true; - return 0; + return false; } - -int +bool locate_song_match(const struct song *song, int num_items, const struct locate_item *items) { for (int i = 0; i < num_items; i++) if (!locate_tag_match(song, items[i].tag, items[i].needle)) - return 0; + return false; - return 1; + return true; } diff --git a/src/locate.h b/src/locate.h index 92bedd90e..7442cb3a3 100644 --- a/src/locate.h +++ b/src/locate.h @@ -20,6 +20,7 @@ #define MPD_LOCATE_H #include +#include #define LOCATE_TAG_FILE_TYPE TAG_NUM_OF_ITEM_TYPES+10 #define LOCATE_TAG_ANY_TYPE TAG_NUM_OF_ITEM_TYPES+20 @@ -50,11 +51,11 @@ locate_item_list_free(int count, struct locate_item *array); void locate_item_free(struct locate_item *item); -int +bool locate_song_search(const struct song *song, int numItems, const struct locate_item *items); -int +bool locate_song_match(const struct song *song, int numItems, const struct locate_item *items);