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



Rockbox mail archive

Subject: vorbis comment patch

vorbis comment patch

From: Pedro Vasconcelos <pbv_at_st-andrews.ac.uk>
Date: 2005-07-09

Hi all,

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.

Cheers,

Pedro

_______________________________________________
http://cool.haxx.se/mailman/listinfo/rockbox

Received on Sat Jul 9 16:01:40 2005

Page was last modified "Jan 10 2012" The Rockbox Crew
aaa