[FFmpeg-devel] [PATCH] Add to AVCodec a long_name field

[Michael Niedermayer]

On Tue, Apr 15, 2008 at 12:13:38PM +0200, Stefano Sabatini wrote:
> On date Tuesday 2008-04-15 11:47:41 +0200, Michael Niedermayer encoded:
> > On Tue, Apr 08, 2008 at 04:02:48PM +0200, Stefano Sabatini wrote:
> > [...]
> > > Index: libavcodec/avcodec.h
> > > ===================================================================
> > > --- libavcodec/avcodec.h	(revision 12769)
> > > +++ libavcodec/avcodec.h	(working copy)
> > > @@ -30,7 +30,7 @@
> > >  #include "libavutil/avutil.h"
> > >  
> > >  #define LIBAVCODEC_VERSION_MAJOR 51
> > > -#define LIBAVCODEC_VERSION_MINOR 54
> > > +#define LIBAVCODEC_VERSION_MINOR 55
> > >  #define LIBAVCODEC_VERSION_MICRO  0
> > >  
> > >  #define LIBAVCODEC_VERSION_INT  AV_VERSION_INT(LIBAVCODEC_VERSION_MAJOR, \
> > > @@ -2235,6 +2235,7 @@
> > >      void (*flush)(AVCodecContext *);
> > >      const AVRational *supported_framerates; ///array of supported framerates, or NULL if any, array is terminated by {0,0}
> > >      const enum PixelFormat *pix_fmts;       ///array of supported pixel formats, or NULL if unknown, array is terminanted by -1
> > > +    const char *long_name;
> > >  } AVCodec;
> > 
> > missing doxygen comment, all field in public structs should have one
> > explaining what the contain
> 
> Is it OK like this?

well this isnt the best way to describe the field but, yes

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

If you really think that XML is the answer, then you definitly missunderstood
the question -- Attila Kinali
-------------- 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/20080415/c687d0bb/attachment.pgp>