Rockbox

Attached to Project: Rockbox
Opened by billnapier - 2003-01-22
Last edited by zagor - 2003-03-10

The patch allows the id3v2 parser to ignore any duplicate
tags and only accept the first tag it sees. According to the
Id3v2 spec, this seems to be an appropriate behavior. The
spec indicates that you SHOULD put the most important
tags first in the list (but doesn’t require any order). I
thought it would then be reasonable that the parser accepts
the first tag it sees and rejects any additional tags.

For example:
A few mp3’s I have have both a TIT2 and TT2 tag. I don’t
know why. Currently we pick the last tag seen (in these
cases TT2). This patch reverses it to use the first tag seen
(in these cases, the TIT2 which happens to have the right
information in it).