leosongwei / mutagen

Automatically exported from code.google.com/p/mutagen
GNU General Public License v2.0
0 stars 0 forks source link

Some shitty encoders write corrupt Ogg/Vorbis files #13

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
Mutagen relies on the string "OggS" being in the exact beginning of the
file to parse the metadata.

This is bad as some encoders (grip, for example) put a ID3v2 tag before the
vorbis comments by default, so as to allow shitty portable audio players to
read the metadata.

Anyway, the Vorbis I spec does not specify that the file should start with
"OggS". At least I couldn't find it ;)

And vorbis-tools read and edit correctly the comments that have an offset,
so I think it's cool to support them.

Original issue reported on code.google.com by maltu...@gmail.com on 17 Jul 2009 at 12:41

GoogleCodeExporter commented 9 years ago
Grip is a piece of shit. You should disable that option. I am complete unaware 
of
portable players that support decoding Ogg Vorbis but do not support proper 
Vorbis
metadata.

The Ogg standard (http://www.xiph.org/vorbis/doc/framing.html) clearly says 
that all
pages must start with OggS, and the Vorbis standard
(http://www.xiph.org/vorbis/doc/Vorbis_I_spec.html#x1-127000A.1) explicitly 
says the
encapsulating Ogg stream may not have any additional header information to be
consider an Ogg/Vorbis I file.

Original comment by joe.wreschnig@gmail.com on 20 Jul 2009 at 1:39