Rockbox.org home
release
dev builds
extras
themes manual
wiki
device status forums
mailing lists
IRC bugs
patches
dev guide



Rockbox mail archive

Subject: Re: vorbis comment patch

Re: vorbis comment patch

From: Magnus Holmgren <lear_at_algonet.se>
Date: Sun, 10 Jul 2005 22:35:09 +0200 (MEST)

On Sat, 9 Jul 2005, Pedro Vasconcelos wrote:

> I found an error in the playback parsing of vorbis comments: for files
> with non-numeric data in the DATE and TRACKNUMBER tags the parser would
> corrupt the remaing tags. I'm attaching a patch that corrects this and
> also adds the parsing of the COMPOSER tag (useful for classical music
> fans like me).
>
> The patch also removes the test that enforces numeric fields to contain
> only digits. What it now does is just verify that the field begins with
> a digit; this means that a tag like 'TRACKNUMBER=10/12' will be accepted
> (and tracknumber set to 10). This behaviour is more tolerant of slight
> incorrections in tagging and avoid the extra tests.
>
> If and when everyone is happy with this I can commit it to the CVS.

No complaint about your patch, but maybe I should've commited my patch (on
the tracker btw) after all. It "includes" your fixes... :) Btw, there is a
track_string field in the mp3entry struct, which is checked before the
numeric field. Just copy the string there...

Magnus Holmgren


_______________________________________________
http://cool.haxx.se/mailman/listinfo/rockbox
Received on 2005-07-10

Page template was last modified "Tue Sep 7 00:00:02 2021" The Rockbox Crew -- Privacy Policy