Posted it to trac bug tracking at <a href="https://trac.xiph.org/ticket/1308">https://trac.xiph.org/ticket/1308</a> and to <a href="http://wiki.xiph.org/index.php/RFC_3533_Errata">http://wiki.xiph.org/index.php/RFC_3533_Errata</a>.<br>
<br>Cheers,<br>Silvia.<br><br><div class="gmail_quote">On Tue, Feb 26, 2008 at 10:42 AM, Conrad Parker <<a href="mailto:conrad@metadecks.org">conrad@metadecks.org</a>> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d">On 26/02/2008, Ralph Giles <<a href="mailto:giles@xiph.org">giles@xiph.org</a>> wrote:<br>
> On 25-Feb-08, at 9:34 AM, <a href="mailto:xiphmont@xiph.org">xiphmont@xiph.org</a> wrote:<br>
><br>
> > I'd say both; libogg should ignore the incomplete packet or flag a<br>
> > structural error, and the spec should be clarified.<br>
><br>
> I guess libogg should return -1 if you try to ogg_stream_packetout()<br>
> after the eos packet.<br>
><br>
> The eos flag pretty clearly applies to the *page* not the packet, so<br>
> I think how that gets translated into the corresponding ogg_packet<br>
> structure is a matter for the libogg api documentation, not the spec.<br>
<br>
</div>ok. If the correct behaviour should be to ignore the incomplete<br>
packet, that should also be clarified in the spec.<br>
<br>
I'd find it useful if it was not considered a structural error, as it<br>
allows us to very simply chop files apart and put them back together.<br>
The eos flag on the page is then a very clear way of saying to ignore<br>
the extra data during decode.<br>
<br>
As far as the libogg API is concerned, I reckon it should also set<br>
op->e_o_s on the last completed packet on a page marked eos.<br>
<br>
cheers,<br>
<font color="#888888"><br>
Conrad.<br>
</font><div><div></div><div class="Wj3C7c">_______________________________________________<br>
ogg-dev mailing list<br>
<a href="mailto:ogg-dev@xiph.org">ogg-dev@xiph.org</a><br>
<a href="http://lists.xiph.org/mailman/listinfo/ogg-dev" target="_blank">http://lists.xiph.org/mailman/listinfo/ogg-dev</a><br>
</div></div></blockquote></div><br>