vorbiscomments: Remove support for non-standard "album artist" tags.

MPD used both "album artist" and "albumartist" tags and mapped them to one tag. This could lead to issues, if a file had both tags, causing MPD to send a list of albumartists instead of a single one. Since "album artist" is not a standard tag anyway and even its originators started to use the proper alternative, its time to say goodbye!

vorbiscomments: Remove support for non-standard "album artist" tags.
7d9aa5b7 · Rasmus Steinke · 43267dc8 · 7d9aa5b7 · 7d9aa5b7
Commit 7d9aa5b7 authored Dec 12, 2014 by Rasmus Steinke
Hide whitespace changes
Inline Side-by-side

Showing with 3 additions and 2 deletions

NEWS NEWS +3 -1

XiphTags.cxx src/decoder/plugins/XiphTags.cxx +0 -1

No files found.
--- a/NEWS
+++ b/NEWS
@@ -4,7 +4,9 @@ ver 0.20 (not yet released)
  - "search"/"find" have a "window" parameter
  - report song duration with milliseconds precision
 * tags
-  - ape: drop support for non-standard tag "album artist"
+  - ape, ogg: drop support for non-standard tag "album artist"
+    affected filetypes: vorbis, flac, opus & all files with ape2 tags
+    (most importantly some mp3s)
 * decoder
  - ffmpeg: support ReplayGain and MixRamp
  - ffmpeg: support stream tags

--- a/src/decoder/plugins/XiphTags.cxx
+++ b/src/decoder/plugins/XiphTags.cxx
@@ -27,7 +27,6 @@
 const struct tag_table xiph_tags[] = {
 	{ "tracknumber", TAG_TRACK },
 	{ "discnumber", TAG_DISC },
-	{ "album artist", TAG_ALBUM_ARTIST },
 	{ "description", TAG_COMMENT },
 	{ nullptr, TAG_NUM_OF_ITEM_TYPES }
 };