[FFmpeg-devel] [PATCH] Eliminate -dc option from ffmpeg.c options

Stefano Sabatini stefano.sabatini-lala
Sat Aug 9 19:58:09 CEST 2008


On date Saturday 2008-08-09 19:18:21 +0200, Michael Niedermayer encoded:
> On Sat, Aug 09, 2008 at 01:44:10PM +0200, Stefano Sabatini wrote:
> > On date Saturday 2008-08-09 04:28:14 +0200, Michael Niedermayer encoded:
> > > On Fri, Aug 08, 2008 at 02:48:57PM -0700, Baptiste Coudurier wrote:
> > > > Hi,
> > > > 
> > > > Stefano Sabatini wrote:
> > > > > Hi all,
> > > > > 
> > > > > this seems unnecessary since it may be set using AVOptions.
> > > > > 
> > > > > Also I honestly cannot understand the -8 logic, but it seems to me
> > > > > plain wrong/misleading.
> > > > > 
> > > > 
> > > > Not at all, everybody express MPEG-2 (at least) DC precision in bits, so
> > > > you say 9/10/11 bits for dc 1/2/3.
> > > > 
> > > > This change command line usage and Im against it for now, some more
> > > > advanced AVOption system like eval() possibility would be better I think.
> > > 
> > > hmm, i think having command line options and internal variables do very
> > > different things could be confusing.
> > > 
> > > We could just make intra_dc_precision have 8,9,10,11 as value instead of
> > > 1/2/3 this would IMHO be the cleanest solution but would have to be
> > > done under #if VERSION with the next major version bump ...
> > 
> > Like this?
> 
> no
> 
> the parameter should take values in the range of 8..12 nor 0..4

libavcodec/utils.c currently takes the value using intra_dc_precision
values (ranging from 0 to 4), I left that in order to avoid to change
the code in the rest of libavcodec.

The new option in utils.c options simply uses another name, which is
the same used for the field (as I think it is recommended), what I'm
trying to avoid with this patch is having two options in ffmpeg with
the same name but with two different meanings.

The ffmpeg.c patch clarifies the meaning of the parameter for the dc
option (bits_of_precision), while the utils.c one simply changes the
name of the now called dc option to avoid clashes with that defined in
ffmpeg.c, this way the user may use the option which he prefers.

Sorry if I misunderstood your comment.

Regards.
-- 
FFmpeg = Foolish Fiendish Merciless Problematic Explosive God




More information about the ffmpeg-devel mailing list