Bug fixes: Artist was skipped sometimes. Frequency, mpeg version & layer was never copied.

git-svn-id: svn://svn.rockbox.org/rockbox/trunk@3441 a1c6a512-1295-4272-9138-f99709370657

1 files changed, 4 insertions, 1 deletions

diff --git a/firmware/id3.c b/firmware/id3.c
index 96b660658d..b761f50b86 100644
--- a/firmware/id3.c
+++ b/firmware/id3.c
@@ -346,7 +346,7 @@ static void setid3v2title(int fd, struct mp3entry *entry)
             bufferpos += bytesread + 2;
             size -= bytesread;
         }
-        else if (!entry->artist &&
+        else if (!entry->title &&
                  (!strncmp(header, "TIT2", strlen("TIT2")) || 
                   !strncmp(header, "TT2", strlen("TT2")))) {
             bytesread = read(fd, buffer + bufferpos, framelen);
@@ -475,6 +475,9 @@ static int getsonglength(int fd, struct mp3entry *entry)
     bytecount += entry->id3v2len;
 
     entry->bitrate = info.bitrate;
+    entry->frequency = info.frequency;
+    entry->version = info.version;
+    entry->layer = info.layer;
 
     /* If the file time hasn't been established, this may be a fixed
        rate MP3, so just use the default formula */