[FFmpeg-devel] [PATCH] Add support for lavfi filter long names

[Stefano Sabatini]

On date Tuesday 2009-10-20 01:20:38 +0200, Michael Niedermayer encoded:
> On Mon, Oct 19, 2009 at 09:25:20PM +0200, Stefano Sabatini wrote:
> > Hi,
> > as suggested by Diego.
> > 
> > Regards.
> > -- 
> > FFmpeg = Free and Frenzy Martial Proud Erratic Genius
> 
> >  avfilter.h |    8 +++++++-
> >  1 file changed, 7 insertions(+), 1 deletion(-)
> > a691c8bbae8e961f729fc111b210423ef0bf3436  lavfi-add-long-name.patch
> > Index: libavfilter/avfilter.h
> > ===================================================================
> > --- libavfilter/avfilter.h	(revision 20314)
> > +++ libavfilter/avfilter.h	(working copy)
> > @@ -23,7 +23,7 @@
> >  #define AVFILTER_AVFILTER_H
> >  
> >  #define LIBAVFILTER_VERSION_MAJOR  1
> > -#define LIBAVFILTER_VERSION_MINOR  3
> > +#define LIBAVFILTER_VERSION_MINOR  4
> >  #define LIBAVFILTER_VERSION_MICRO  0
> >  
> >  #define LIBAVFILTER_VERSION_INT AV_VERSION_INT(LIBAVFILTER_VERSION_MAJOR, \
> > @@ -426,6 +426,12 @@
> >      AVFilterLink **outputs;         ///< array of pointers to output links
> >  
> >      void *priv;                     ///< private data for use by the filter
> > +
> > +    /**
> > +     * Descriptive name for the filter. You should use the
> > +     * NULL_IF_CONFIG_SMALL() macro to define it.
> > +     */
> > +    const char *long_name;
> 
> somthing tells me that description would be better than long_name
> like in "(Filter that )flips the given image vertically"

Absolutely, patch updated.

> but anyway iam fine with this, the color is for others to fight over

[...]

Regards.
-- 
FFmpeg = Fierce and Fundamentalist Mythic Patchable Egregious Gangster
-------------- next part --------------
A non-text attachment was scrubbed...
Name: lavfi-add-description.patch
Type: text/x-diff
Size: 809 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20091021/eaca6edc/attachment.patch>