Hi Ruud van Asseldonk,

> On Jun 11, 2017, at 7:24 AM, Ruud van Asseldonk <d...@veniogames.com> wrote:
> 
>>> I'm jumping in on this thread to make a few remarks about the spec. I
>>> implemented a FLAC decoder by only looking at the spec, and I have a few
>>> notes that would have saved me a lot of time if the spec had mentioned
>>> them. They are obvious in hindsight, of course.
>>> 
>>> * If the channel assignment includes a difference channel, then the
>>> subframe for that channel has one extra bit per sample in order to
>>> encode the difference.
>>> 
>>> * The number of bits per sample for a subframe, is the number of bits
>>> per sample of the frame, minus the number of wasted bits per sample of
>>> the subframe (and possibly plus one for a difference channel).
>>> 
>>> I hope this helps future implementers.
>> 
>> I would love to see a patch against the documentation for this.
>> 
>> Erik
> 
> Here you go.
> 
> Kind regards,
> 
> Ruud van Asseldonk
> <0001-doc-Add-notes-about-subframe-sample-size.patch>_______________________________________________

Do you mind to submit this patch as well to cel...@ietf.org 
<mailto:cel...@ietf.org> (info at https://www.ietf.org/mailman/listinfo/cellar 
<https://www.ietf.org/mailman/listinfo/cellar>). That working group is 
currently working on the Flac format documentation in 
https://github.com/privatezero/flac_markdown 
<https://github.com/privatezero/flac_markdown> and 
https://datatracker.ietf.org/wg/cellar/documents/ 
<https://datatracker.ietf.org/wg/cellar/documents/>.
Dave Rice

_______________________________________________
flac-dev mailing list
flac-dev@xiph.org
http://lists.xiph.org/mailman/listinfo/flac-dev

Reply via email to