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

Michael Niedermayer michaelni
Tue Oct 20 01:20:38 CEST 2009


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"

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

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

Those who are too smart to engage in politics are punished by being
governed by those who are dumber. -- Plato 
-------------- 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/20091020/f3cd2572/attachment.pgp>



More information about the ffmpeg-devel mailing list