[FFmpeg-devel] Behaviour of liba52 decoder

[Måns Rullgård]

Rich Felker <dalias at aerifal.cx> writes:

> On Thu, Jan 10, 2008 at 12:57:33PM +0100, Thorsten Jordan wrote:
>> Michael Niedermayer schrieb:
>> > Hi
>> > 
>> >> any comments?
>> > 
>> > a52_decode_frame() looks buggy yes ...
>> > i suggest you rewrite it ...
>> > 
>> > it should roughly look like
>> > 
>> > read header
>> > for(all 6 block)
>> >     decode block
>> this would be ok if one could demand that full frames are given always.
>
> All lavc decoders are entitled to assume this.

I'm surprised nobody has mentioned the fact that we now have a native
AC3 decoder, so there is no longer any need for the liba52 wrapper.
Is there some reason I'm missing why it's still there?

-- 
M?ns Rullg?rd
mans at mansr.com