<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class="">Hi Ruud van Asseldonk,</div><br class=""><div><blockquote type="cite" class=""><div class="">On Jun 11, 2017, at 7:24 AM, Ruud van Asseldonk <<a href="mailto:dev@veniogames.com" class="">dev@veniogames.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div class=""><blockquote type="cite" class=""><blockquote type="cite" class="">I'm jumping in on this thread to make a few remarks about the spec. I<br class="">implemented a FLAC decoder by only looking at the spec, and I have a few<br class="">notes that would have saved me a lot of time if the spec had mentioned<br class="">them. They are obvious in hindsight, of course.<br class=""><br class="">* If the channel assignment includes a difference channel, then the<br class="">subframe for that channel has one extra bit per sample in order to<br class="">encode the difference.<br class=""><br class="">* The number of bits per sample for a subframe, is the number of bits<br class="">per sample of the frame, minus the number of wasted bits per sample of<br class="">the subframe (and possibly plus one for a difference channel).<br class=""><br class="">I hope this helps future implementers.<br class=""></blockquote><br class="">I would love to see a patch against the documentation for this.<br class=""><br class="">Erik<br class=""></blockquote><br class="">Here you go.<br class=""><br class="">Kind regards,<br class=""><br class="">Ruud van Asseldonk<br class=""><span id="cid:14A0680F-39AC-4220-9D5E-B9E59D9D7BB8@nyc.rr.com"><0001-doc-Add-notes-about-subframe-sample-size.patch></span>_______________________________________________<br class=""></div></div></blockquote><br class=""></div><div>Do you mind to submit this patch as well to <a href="mailto:cellar@ietf.org" class="">cellar@ietf.org</a> (info at <a href="https://www.ietf.org/mailman/listinfo/cellar" class="">https://www.ietf.org/mailman/listinfo/cellar</a>). That working group is currently working on the Flac format documentation in <a href="https://github.com/privatezero/flac_markdown" class="">https://github.com/privatezero/flac_markdown</a> and <a href="https://datatracker.ietf.org/wg/cellar/documents/" class="">https://datatracker.ietf.org/wg/cellar/documents/</a>.</div><div>Dave Rice</div><br class=""></body></html>