[FFmpeg-devel] [PATCH] [2/??] [3/3] Filter graphs - Parser for a graph description

Vitor Sessak vitor1001
Fri Apr 18 08:57:42 CEST 2008


Vitor Sessak wrote:

[...]

> A final question: should I, instead of having as a prototype
> 
> int avfilter_parse_graph(AVFilterGraph *graph, const char *filters,
>                          AVFilterContext *in, int inpad,
>                          AVFilterContext *out, int outpad,
>                          AVClass *log_ctx);
> 
> use
> 
> int avfilter_parse_graph(AVFilterGraph *graph, const char *filters,
>                          AVFilterContext **in, int *inpad,
>                          AVFilterContext **out, int *outpad,
>                          AVClass *log_ctx);
> 
> So I could extend it later to have several inputs/outputs without 
> changing the public API? Are you ok with a NULL-termined array of 
> (AVFilterContext *) for several inputs?

Michael, ping?

(yes, I know you have a lot of patches to review, but maybe you just 
missed this one)

-Vitor




More information about the ffmpeg-devel mailing list