[FFmpeg-devel] [PATCH] cmdutils: include type in filter list.

[Clément Bœsch]

On Thu, Jan 19, 2012 at 01:38:48PM +0100, Nicolas George wrote:
> 
> Signed-off-by: Nicolas George <nicolas.george at normalesup.org>
> ---
> 
> ffmpeg -filters will look like that:
> 
> aconvert         A->A       Convert the input audio to sample_fmt:channel_layout:packed_fmt.
> amerge           AA->A      Merge two audio streams into a single multi-channel stream.
> abuffer          |->A       Buffer audio frames, and make them accessible to the filterchain.
> abuffersink      A->|       Buffer audio frames, and make them available to the end of the filter graph.
> anullsink        A->|       Do absolutely nothing with the input audio.
> deshake          V->V       Stabilize shaky video.
> overlay          VV->V      Overlay a video source on top of the input.
> split            V->VV      Pass on the input to two outputs.
> cellauto         |->V       Create pattern generated by an elementary cellular automaton.
> buffersink       V->|       Buffer video frames, and make them available to the end of the filter graph.
> 
> 
>  cmdutils.c |   55 ++++++++++++++++++++++++++++++++++++-------------------
>  1 files changed, 36 insertions(+), 19 deletions(-)
> 
> diff --git a/cmdutils.c b/cmdutils.c
> index 344e506..7e43d69 100644
> --- a/cmdutils.c
> +++ b/cmdutils.c
> @@ -764,6 +764,18 @@ int opt_formats(const char *opt, const char *arg)
>      return 0;
>  }
>  
> +static char get_media_type_letter(enum AVMediaType type)
> +{
> +    char map[AVMEDIA_TYPE_NB] = {

static const?

> +        [AVMEDIA_TYPE_VIDEO]      = 'V',
> +        [AVMEDIA_TYPE_AUDIO]      = 'A',
> +        [AVMEDIA_TYPE_DATA]       = 'D',
> +        [AVMEDIA_TYPE_SUBTITLE]   = 'S',
> +        [AVMEDIA_TYPE_ATTACHMENT] = 'J',

Stream specifiers use 't' for attachment, maybe 'T' would be appropriate
here too?

[...]

-- 
Clément B.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 490 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120119/705ea473/attachment.asc>