[FFmpeg-devel] [fix-multiple-filter-chain.patch PATCH 3/3] Cosmetics: rename enc -> avctx in opt_input_file(), the variable refer to a *decoder*, the old name was misleading.

Stefano Sabatini stefano.sabatini-lala
Sat Jul 17 13:04:00 CEST 2010


On date Friday 2010-07-09 16:11:12 +0200, Michael Niedermayer encoded:
> On Fri, Jul 09, 2010 at 12:56:24AM +0200, Stefano Sabatini wrote:
> > On date Wednesday 2010-07-07 20:26:15 +0200, Michael Niedermayer encoded:
> > > On Wed, Jul 07, 2010 at 08:03:29PM +0200, Stefano Sabatini wrote:
> > > > ---
> > > >  ffmpeg.c |   44 ++++++++++++++++++++++----------------------
> > > >  1 files changed, 22 insertions(+), 22 deletions(-)
> > > 
> > > if its a decoder iz should be called dec
> > 
> > Sorry the commmit message was wrong, the variable is a decoder
> > *context* not a decoder, in this regards the use of avctx makes sense.
> 
> dec= DEcoderContext then.

That would be inconsistent with other parts of ffmpeg.c (and most
FFmpeg code). Also the more natural interpretation of dec is DECoder.

Regards.



More information about the ffmpeg-devel mailing list