[FFmpeg-devel] [PATCH] enc is not a good name for a decoder context

Michael Niedermayer michaelni
Tue Dec 29 01:21:13 CET 2009


On Mon, Dec 28, 2009 at 11:30:07PM +0100, Stefano Sabatini wrote:
> On date Monday 2009-12-28 23:19:59 +0100, Michael Niedermayer encoded:
> > On Mon, Dec 28, 2009 at 10:56:25PM +0100, Stefano Sabatini wrote:
> > > Hi, as in subject.
> > > 
> > > Regards.
> > > -- 
> > > FFmpeg = Friendly and Foolish Multimedia Portable Ecstatic Geek
> > 
> > >  ffplay.c |   68 +++++++++++++++++++++++++++++++--------------------------------
> > >  1 file changed, 34 insertions(+), 34 deletions(-)
> > > b9e8704744f4cf2e98f3451c6e34c3d75385c30f  ffplay-use-better-name-for-decoder-ctx.patch
> > > Index: ffplay.c
> > > ===================================================================
> > > --- ffplay.c	(revision 20954)
> > > +++ ffplay.c	(working copy)
> > > @@ -1692,50 +1692,50 @@ static void sdl_audio_callback(void *opaque, Uint8
> > >  static int stream_component_open(VideoState *is, int stream_index)
> > >  {
> > >      AVFormatContext *ic = is->ic;
> > > -    AVCodecContext *enc;
> > > +    AVCodecContext *dec;
> > 
> > dec is not a good name for AVCodecContext either
> 
> dec_ctx?
> decoder_ctx?
> codec_ctx?

avctx

> 
> I just followed the convention used in ffmpeg.c...

ffmpeg uses encoders and decoders ffplay ust decoders

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Concerning the gods, I have no means of knowing whether they exist or not
or of what sort they may be, because of the obscurity of the subject, and
the brevity of human life -- Protagoras
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20091229/b6112952/attachment.pgp>



More information about the ffmpeg-devel mailing list