[FFmpeg-devel] [PATCH] FLAC parser

[Justin Ruggles]

Michael Niedermayer wrote:

> On Mon, Jul 19, 2010 at 12:55:05AM +0200, Michael Chinen wrote:
>> Hi,
>>
>> This FLAC parser takes the suggestions from a thread from another FLAC
>> parser patch submitted by Jason Ruggles in March 2009[1].
>> Currently it stores 20 headers (8 bit crc verified) and finds all
>> possible (16 bit footer) crc-verified sequences within a neighbor
>> distance of 4.
>> It penalizes sequences that have changes in sample rate, bit depth,
>> and channel arrangement.
>> The settings probably need some twiddling.
> 
> you should check the frame number too (unless encoders dont set it like
> ours does)
> without the frame number there can be ambigous cases with finite probability
> even as the number of frames considered goes to infinity

Good thought.  Each frame header has either the frame number or sample
number (depending on whether it has variable or fixed block size).  But
either one could be tracked and checked.

-Justin