[Ffmpeg-devel] [PATCH] fix -ab default

Baptiste Coudurier baptiste.coudurier
Wed Mar 7 00:36:01 CET 2007


Michael Niedermayer wrote:
> Hi
> attached patch adds a new avcodec_get_context_defaults2() and 
> avcodec_alloc_context2() which take an additional CodecType as argument
> it adds a av_opt_set_defaults2() which has a flags parameter so only
> specific defaults are used
> and it adds a -ab to AVOption whos flags specify that it only applies for
> audio and -b only applies to video
> and last it makes ffmpeg.c use the new code
> all that together fixes the -ab default (note the recent changes to
> command line option handling did not truly introduce this bug it was
> rather hidden by the old code ...)
> so everyone who flamed please now read and comment! that is
> comment now not after i applied it!
> is CodecType a good choice as argument for the default setting function?
> i choose it as it was slightly easier, alternatives are AVCodec and
> CodecID, the advantage of the later 2 would be that we would move
> toward being able to have per codec defaults which could come in handy
> for x264

Well, since we cannot afford to change API oftenly, would it be hard to
use CodecID now ? Im sure some people would be happy to start tweaking
x264 default settings.

Baptiste COUDURIER                              GnuPG Key Id: 0x5C1ABAAA
SMARTJOG S.A.                                    http://www.smartjog.com
Key fingerprint                 8D77134D20CC9220201FC5DB0AC9325C5C1ABAAA
Phone: +33 1 49966312

More information about the ffmpeg-devel mailing list