On 18/08/13 17:40, Jan Gerber wrote:
>+            // Do not set incomplete for empty packets.
>+            // Together with the code in ogg_read_page
>+            // that discards all continuation of empty packets
>+            // we would get an infinite loop.
>+            os->incomplete = !!os->psize;

Basically we set a packet as complete once we get an empty packet.

Not sure if won't open us to different funny behaviours but looks safer
than keep eating packets till the end of time.

Which is the semantic of empty packet in ogg though?

The patch seems fine to me, maybe with the first hunk dropped.

lu
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to