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

[Stefano Sabatini]

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?

BTW, "///" comments aren't recognized by Doxygen and "terminanted" is
a typo (patches incoming for that too).

Best regards.
-- 
FFmpeg =  Fiendish and Fucked MultiPurpose EniGma
-------------- next part --------------
A non-text attachment was scrubbed...
Name: add-avcodec-long-name-field-01.patch
Type: text/x-diff
Size: 862 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20080415/709ce572/attachment.patch>