[MPlayer-dev-eng] [PATCH] external liba52 support

Diego Biurrun diego at biurrun.de
Tue Sep 16 09:43:26 CEST 2008


On Mon, Sep 15, 2008 at 06:34:39PM +0200, Dominik 'Rathann' Mierzejewski wrote:
> On Saturday, 13 September 2008 at 23:27, Dominik 'Rathann' Mierzejewski wrote:
> > On Saturday, 13 September 2008 at 23:24, Dominik 'Rathann' Mierzejewski wrote:
> > > On Saturday, 13 September 2008 at 22:11, Diego Biurrun wrote:
> > > > On Sat, Sep 13, 2008 at 04:00:26PM +0200, Dominik 'Rathann' Mierzejewski wrote:
> > > > > On Saturday, 30 August 2008 at 01:00, Dominik 'Rathann' Mierzejewski wrote:
> > > > > > On Saturday, 30 August 2008 at 00:48, Diego Biurrun wrote:
> > > > > > > On Fri, Aug 22, 2008 at 02:14:23AM +0200, Dominik 'Rathann' Mierzejewski wrote:
> > > > > > > > Here are two patches which add external liba52-0.7.4 support.
> > > > > > > 
> > > > > > > This is very welcome, thanks.
> > > > > > > 
> > > > > > > > Patch1: conditionalizes enabling of some acceleration because
> > > > > > > > liba52-0.7.4 doesn't support all that MPlayer's copy does.
> > > > > > > > Patch2: the main patch, depends on Patch1
> > > > > > > > 
> > > > > > > > Tested with --disable-liba52-internal and without, seems to be working fine.
> > > > > 
> > > > > Updated patches against current SVN attached.
> > > > > 
> > > > > OK to apply?
> > > > 
> > > > Some small comments below, the rest is OK from me.
> > 
> > I'll apply tomorrow if there are no further comments.
> 
> Applied. Do you think it's worth a Changelog entry?

Sure, why not..

Diego



More information about the MPlayer-dev-eng mailing list