[FFmpeg-devel] Behaviour of liba52 decoder

[Rich Felker]

On Thu, Jan 10, 2008 at 06:28:52PM +0000, M?ns Rullg?rd wrote:
> 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?

Good question. I'd be happy to see it removed.

Rich