On 09/22/2012 09:16 AM, Justin Ruggles wrote:
> On 09/22/2012 08:58 AM, Luca Barbato wrote:
>> diff --git a/libavformat/oggparseflac.c b/libavformat/oggparseflac.c
>> index 9860a0e..f59b400 100644
>> --- a/libavformat/oggparseflac.c
>> +++ b/libavformat/oggparseflac.c
>> @@ -88,11 +88,13 @@ old_flac_header (AVFormatContext * s, int idx)
>> const struct ogg_codec ff_flac_codec = {
>> .magic = "\177FLAC",
>> .magicsize = 5,
>> - .header = flac_header
>> + .header = flac_header,
>> + .nb_header = 2,
>> };
>>
>> const struct ogg_codec ff_old_flac_codec = {
>> .magic = "fLaC",
>> .magicsize = 4,
>> - .header = old_flac_header
>> + .header = old_flac_header,
>> + .nb_header = 0,
>> };
>
> The current FLAC mapping is the one that clearly allows any number of
> headers. I'm not sure what the "old" FLAC mapping allows because it's
> not specified anywhere AFAIK.
Nevermind. I didn't see that you changed == to <
LGTM
-Justin
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
