[FFmpeg-soc] [soc]: r2396 - aacenc/aacenc.c

Diego Biurrun diego at biurrun.de
Sat Jun 7 13:48:21 CEST 2008


On Sat, Jun 07, 2008 at 02:01:47PM +0300, Kostya wrote:
> On Sat, Jun 07, 2008 at 11:56:14AM +0200, Diego Biurrun wrote:
> > On Sat, Jun 07, 2008 at 11:48:02AM +0200, kostya wrote:
> > > 
> > > Log:
> > > Rename context variable for consistency
> > > 
> > > --- aacenc/aacenc.c	(original)
> > > +++ aacenc/aacenc.c	Sat Jun  7 11:48:02 2008
> > > @@ -62,14 +62,14 @@ static int put_audio_specific_config(AVC
> > >  static int aac_encode_init(AVCodecContext *avctx)
> > >  {
> > > -    AACEncContext *c = avctx->priv_data;
> > > +    AACEncContext *s = avctx->priv_data;
> > 
> > Is 's' really a better name for a context than 'c'?
> 
> Have I said anything about it being a better name?
> I'm equally used to "c" and "s" for codec context names.
> But the majority of names here was "s", so I decided to
> stick to it.

FTLIW, I prefer 'c' or 'ctx', but use whatever floats your boat.

Diego



More information about the FFmpeg-soc mailing list