ffmeg: added support for the tags comment, genre, year

ffmpeg_tag_internal() does not look for a few tags that mpd supports. Most noteably: comment -> TAG_ITEM_COMMENT -> Description genre -> TAG_ITEM_GENRE -> WM/Genre (not WM/GenreID) year -> TAG_ITEM_DATE -> WM/Year I *think* that this is the last of the tags that AVFormatContext() in ffmpeg supports that mpd also uses.

ffmeg: added support for the tags comment, genre, year
efb04532 · David Horn · Max Kellermann · 32d6d449 · efb04532 · efb04532
Commit efb04532 authored Jan 30, 2009 by David Horn Committed by Max Kellermann Jan 30, 2009
Hide whitespace changes
Inline Side-by-side

Showing with 12 additions and 0 deletions

NEWS NEWS +2 -0

ffmpeg_plugin.c src/decoder/ffmpeg_plugin.c +10 -0

No files found.
--- a/NEWS
+++ b/NEWS
@@ -35,6 +35,8 @@ ver 0.15 - (200?/??/??)


 ver 0.14.2 (2009/??/??)
+* decoders:
+  - ffmpeg: added support for the tags comment, genre, year
 * audio outputs:
  - jack: allocate ring buffers before connecting
  - jack: clear "shutdown" flag on reconnect

--- a/src/decoder/ffmpeg_plugin.c
+++ b/src/decoder/ffmpeg_plugin.c
@@ -347,6 +347,16 @@ static bool ffmpeg_tag_internal(struct ffmpeg_context *ctx)
 		tag_add_item(tag, TAG_ITEM_TRACK, buffer);
 	}

+	if (f->comment[0])
+		tag_add_item(tag, TAG_ITEM_COMMENT, f->comment);
+	if (f->genre[0])
+		tag_add_item(tag, TAG_ITEM_GENRE, f->genre);
+	if (f->year > 0) {
+		char buffer[16];
+		snprintf(buffer, sizeof(buffer), "%d", f->year);
+		tag_add_item(tag, TAG_ITEM_DATE, buffer);
+	}
+
 	return true;
 }