aboutsummaryrefslogtreecommitdiffstats
path: root/src/decoder/oggvorbis_plugin.c
diff options
context:
space:
mode:
authorMax Kellermann <max@duempel.org>2009-01-14 23:09:31 +0100
committerMax Kellermann <max@duempel.org>2009-01-14 23:09:31 +0100
commit5a26c949bbdc5791d6884131ab8b18295f5b6f9e (patch)
treec0300563442059f704374d6f00f00ee17bb8d570 /src/decoder/oggvorbis_plugin.c
parentb5cadc9c04c6a37c81475d5c8240a4e485f73555 (diff)
downloadmpd-5a26c949bbdc5791d6884131ab8b18295f5b6f9e.tar.gz
mpd-5a26c949bbdc5791d6884131ab8b18295f5b6f9e.tar.xz
mpd-5a26c949bbdc5791d6884131ab8b18295f5b6f9e.zip
oggvorbis: always allocate a tag object
Always allocate a new tag object before parsing the vorbis comments; free it when it turns out to be empty. This simplifies the code a bit.
Diffstat (limited to '')
-rw-r--r--src/decoder/oggvorbis_plugin.c22
1 files changed, 14 insertions, 8 deletions
diff --git a/src/decoder/oggvorbis_plugin.c b/src/decoder/oggvorbis_plugin.c
index 0b8542f07..60ad0d07c 100644
--- a/src/decoder/oggvorbis_plugin.c
+++ b/src/decoder/oggvorbis_plugin.c
@@ -37,6 +37,8 @@
#endif /* HAVE_TREMOR */
#include <glib.h>
+
+#include <assert.h>
#include <errno.h>
#include <stdlib.h>
#include <unistd.h>
@@ -140,11 +142,13 @@ static const char *VORBIS_COMMENT_TRACK_KEY = "tracknumber";
static const char *VORBIS_COMMENT_DISC_KEY = "discnumber";
static bool
-vorbis_parse_comment(char *comment, enum tag_type tag_type,
- struct tag ** tag)
+vorbis_parse_comment(struct tag *tag, char *comment, enum tag_type tag_type)
{
const char *needle;
unsigned int len;
+
+ assert(tag != NULL);
+
switch (tag_type) {
case TAG_ITEM_TRACK:
needle = VORBIS_COMMENT_TRACK_KEY;
@@ -158,10 +162,7 @@ vorbis_parse_comment(char *comment, enum tag_type tag_type,
len = strlen(needle);
if (strncasecmp(comment, needle, len) == 0 && *(comment + len) == '=') {
- if (!*tag)
- *tag = tag_new();
-
- tag_add_item(*tag, tag_type, comment + len + 1);
+ tag_add_item(tag, tag_type, comment + len + 1);
return true;
}
@@ -172,17 +173,22 @@ vorbis_parse_comment(char *comment, enum tag_type tag_type,
static struct tag *
vorbis_comments_to_tag(char **comments)
{
- struct tag *tag = NULL;
+ struct tag *tag = tag_new();
while (*comments) {
int j;
for (j = TAG_NUM_OF_ITEM_TYPES; --j >= 0;) {
- if (vorbis_parse_comment(*comments, j, &tag))
+ if (vorbis_parse_comment(tag, *comments, j))
break;
}
comments++;
}
+ if (tag_is_empty(tag)) {
+ tag_free(tag);
+ tag = NULL;
+ }
+
return tag;
}