[ogg-dev] [PATCH] liboggz: Fix Dirac bitstream parsing

David Flynn davidf+nntp at woaf.net
Tue Nov 4 14:41:48 PST 2008


On 2008-11-04, Ralph Giles <giles at xiph.org> wrote:
> On Tue, Nov 4, 2008 at 12:56 AM, David Flynn <davidf+nntp at woaf.net> wrote:
>
>> Quite a while ago, the specification was modified to change the
>> way the nondefault scan format was signalled; this patch fixes that.
>
> Thanks for the patch. I've tried to fix things as discussed on irc.
> Please review r3762.

That looks fine.

..david

> -    {50,1}, {60000,1001}, {60,1}, {15000,1001}, {25,2},
> +    {50,1}, {60000,1001}, {60,1}, {15000,1001}, {25,2}

Interesting -- i usually leave the final `,' there so that any future
patch that appends something to the list doesn't have to modify the
previous line to insert the ','

..david



More information about the ogg-dev mailing list