[Vorbis-dev] intro + specification remarks + some questions

[Bram - Splice]

Hello  again,


> Yes, this wording is... tough... but so is the encoding optimization
<snip>
> immediately preceeding it, if vector [v] was sorted)

Thanks! Just what I needed...

>> low_neighbor(v, x)
>> {
>>      max_pos = 0;
>>
>>      for (n=0...x-1)
>>          if (v[n] < v[x])
>>              if (v[n] >= v[max_pos])
>>                  max_pos = n;
>>
>>      return max_pos;
>> }
> 
> yes.

Perhaps this -and the equivalent for high_neighbor- could go into the specs?

>> I.e. a bit more highlevel structure to the docs would be cool and
>> provide a lot of clarity. The lowlevel pseudocode is great and highly
>> readable, but I did notice that somewhere halfway (after decoding the
>> headers) the style changed quite a bit leading me to suspect two authors
>> at work.
> 
> The original spec was written entirely by me.  I do recall needing to
> shift style slightly to capture a concept that I couldn't encode in
 > the original style, but I thought I'd gone back and updated the
> previous code to all reflect the 'newer' style.

Oh, so I did find *something* but my analysis was off...

> [I've not commented on several other points that are good suggestions]

Nice, it's good to know my comments weren't useless...


  - bram