[Ffmpeg-devel] [PATCH] AVOption defaults sync

Michael Niedermayer michaelni
Thu Sep 14 15:44:53 CEST 2006


Hi

On Thu, Sep 14, 2006 at 02:56:21PM +0200, Panagiotis Issaris wrote:
> Hi,
> 
> Op dinsdag 12 september 2006 22:09, schreef Michael Niedermayer:
> >[...]
> > > The attached patch synchronizes the defaults specified in the AVOptions[] array 
> > > to the defaults specified in the ffmpeg.c global static variabels. I took the
> > > ones specified in ffmpeg.c as the new ones for AVOptions, as this preserves the 
> > > current behavior. Regression tests pass.
> > 
> > looks ok though i think that we should eventually change the default of 
> > 12frame gop and 200kbit its probably not what most users would want
> Does the "eventually" imply that I can commit the patch as is, and
> that the defaults can be modified in a follow-up patch?

yes


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In the past you could go to a library and read, borrow or copy any book
Today you'd get arrested for mere telling someone where the library is




More information about the ffmpeg-devel mailing list