aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2009-01-24 15:27:05 +0100
committerMax Kellermann <max@duempel.org>2009-01-24 15:27:05 +0100
commit535cf5b5c95ab4f26baaabb228a72eebc1cba9c5 (patch)
tree11a6183dac20de77095d7d9b52b8bd5047d3d62d
parent3582977e01d19556dcc7e44a6b71628a9f0a1d02 (diff)
downloadmpd-535cf5b5c95ab4f26baaabb228a72eebc1cba9c5.tar.gz
mpd-535cf5b5c95ab4f26baaabb228a72eebc1cba9c5.tar.xz
mpd-535cf5b5c95ab4f26baaabb228a72eebc1cba9c5.zip
locate: use bool instead of int
Use the C99 bool type instead of integer values (1/0 or 0/-1).
-rw-r--r--src/locate.c61
-rw-r--r--src/locate.h5
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 <stdint.h>
+#include <stdbool.h>
#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);