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

[Reimar Döffinger]

On Sat, Mar 13, 2010 at 05:40:56PM +0100, Diego Biurrun wrote:
> 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.

Well, if you want. I don't like it much, but it's not like I see any vastly
better alternative.