[FFmpeg-devel] [PATCH] mpegaudio float default

Michael Niedermayer michaelni
Mon May 17 15:35:48 CEST 2010


On Mon, May 17, 2010 at 02:13:08PM +0200, Luca Barbato wrote:
> On 05/14/2010 10:58 PM, Michael Niedermayer wrote:
> > On Fri, May 14, 2010 at 09:45:31AM +0200, Luca Barbato wrote:
> >> On 05/13/2010 11:43 PM, Michael Niedermayer wrote:
> >>> Hi
> >>>
> >>> patch below makes mpeg audio default, i will commit this tomorrow
> >>>
> >>
> >>
> >> -do_lavf asf "-acodec mp2" "-r 25"
> >> +do_lavf asf "-acodec mp2" "-acodec mp2" "-r 25"
> > 
> > without that:
> > 
> > --- /home/michael/ffmpeg-svn-tmp/trunk/tests/ref/lavf/asf	2010-03-05 03:23:56.000000000 +0100
> > +++ tests/data/asf.lavf.regression	2010-05-14 15:27:25.000000000 +0200
> > @@ -1,3 +1,3 @@
> >  c544bb40c2f4c09d44318db5228ee499 *./tests/data/lavf/lavf.asf
> >  333375 ./tests/data/lavf/lavf.asf
> > -./tests/data/lavf/lavf.asf CRC=0x9f5ab3e6
> > +./tests/data/lavf/lavf.asf CRC=0x8d6e9cc2
> > make: *** [regtest-asf] Error 1
> 
> How's possible?

This patchs intent is to change the default to float-mp2
That means the vastly unexpected change happens that th default afterwards
is indeed float-mp2.
you asked me to remove the hunk that override the default.

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I have never wished to cater to the crowd; for what I know they do not
approve, and what they approve I do not know. -- Epicurus
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20100517/18b9fe06/attachment.pgp>



More information about the ffmpeg-devel mailing list