[vorbis-dev] Bug in vorbis documentation

[John Ripley]

I've been tinkering (not very seriously) with writing a vorbis (and ogg)
decoder from scratch using only the documentation. I've got as far as
starting to decoding floor 1 without any problems. However, there's a
problem in the floor 1 header decode. Specifically:

22) [floor1_values] = [floor1_values] + [floor1_class_dimensions] element
[i]

hould probably be:

22) [floor1_values] = [floor1_values] + [floor1_class_dimensions] element
[current_class_number]

because floor1_class_dimensions has maximum_class + 1 elements and not
floor1_partitions elements.

Other than that, the documentation for everything else so far is very
precise and complete - I can decode files as far as floor 1 setup
successfully. One small gripe I do have with the pseudo-code style is that
the size of vectors isn't defined before usage. It's not at all necessary,
but it would be nice to check against.

Should I send any further documentation issues here, or into bugzilla?
There's isn't a documentation bug section, but I guess libvorbis would be
the place otherwise.

- John Ripley
--- >8 ----
List archives:  http://www.xiph.org/archives/
Ogg project homepage: http://www.xiph.org/ogg/
To unsubscribe from this list, send a message to 'vorbis-dev-request at xiph.org'
containing only the word 'unsubscribe' in the body.  No subject is needed.
Unsubscribe messages sent to the list will be ignored/filtered.