[MPlayer-dev-eng] [PATCH] Use high-level QuickTime decoding API

[Diego Biurrun]

On Mon, Mar 08, 2010 at 09:56:11AM +0100, Attila Kinali wrote:
> On Sat, 6 Mar 2010 15:26:52 +0100
> "Steinar H. Gunderson" <sgunderson at bigfoot.com> wrote:
> 
> > On Sat, Mar 06, 2010 at 03:06:01PM +0100, Reimar Döffinger wrote:
> > > I can understand it, but I am very much in disfavour of completely throwing
> > > out code that has been tested and proven working for years.
> > 
> > There are basically three options here:
> > 
> >  - Leave this decoder in its current, arguably broken state.
> >  - Implement this change, test as thoroughly as possible (I've already
> >    checked sample files for all QuickTime codecs I could get to work),
> >    and watch out for regressions.
> >  - Make a new, separate QuickTime decoder using the new API, and maintain
> >    both.
> 
> I'm for option 2. There is no point in keeping code if it's not 100% working.

I tend to second that notion.  Steinar appears willing and able to stay
around and fix regressions, so I'm not afraid of new bugs.

Diego