[vorbis-dev] Packet usage

[Dominik Kuhlen]

On Friday 26 March 2004 21:08, Ralph Giles wrote:
> On Fri, Mar 26, 2004 at 07:27:52PM +0100, Dominik Kuhlen wrote:
> > Compare:
> > http://www.xiph.org/ogg/vorbis/doc/vorbis-spec-res.html
> > paragraph: packet decode: step 11 to 21
> > and
> > http://www.xiph.org/ogg/vorbis/doc/Vorbis_I_spec.pdf
> > page 53-54
> >
> > I think between step 18 and 19 in the pdf version
> > is something like:
> > "18.5) if  partition_count < partitions_to_read continue at 4"
> > missing
>
> The pdf version appears to match the reference implementation. (res0.c
> line 643)
Huh, you're right. (I read this part of the spec 100 times and missed always little details :-)
>
> Note that the pdf and compete html versions are more up-to-date than the
> separate page you linked.
I'm not sure whether I understand this: Is the pdf I linked up to date?
If so here are a few minor typos:
-Page 3 paragraph before 1.1.5 :
        "Our own design work indicates the the primary... " should be
        "Our own design work indicates the primary... " 

- Page 28 Step 8 before 4.3.2:
        "window from range [rigth_window_start] ... [n]-1 is zero" should be
        "window from range [right_window_end] ... [n]-1 is zero"

- Page 29 4.3.3 Step1: 
        "... or (channel [vorbis_mapping_angle] element [i]) are set to false," should be
        "... or channel ([vorbis_mapping_angle] element [i]) are set to false,"

-Page 39 top:  Step 6: 
        "[lastval] = zero; " should be
        "[last] = zero;" 

- Page 40: Step 7
        "if ( map element [i] is equal to [iteration_condition] ) continue at step 7" should be
        "if ( map element [i] is equal to [iteration_condition] ) continue at step 5" 
        (not sure about the 5)

- Page 46: Step 21-23
        "hiroom" should be
        "highroom" 

- Page 48: 8.2
        "Reside format partitions each ..." should be
        "Residue format partitions each ..." 
- same paragraph
        "defined for each selected selected classification." should be
        "defined for each selected classification."

- Page 55: 8.6.4
        The numbering is scrambled: 1 2 3 5 6 4 5

- Same page: 8.6.5
        "It may be implemented as an additional stepprior to and" should be
        "It may be implemented as an additional step prior to and" 

- Page 59: 9.2.4.3
        number 15 appears twice

<p>Wow, quite a bunch of typos :-) 
Dominik

--- >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.