[FFmpeg-devel] [PATCH] Factorize code from video_thread() and put it in configure_video_filters().

Stefano Sabatini stefano.sabatini-lala
Tue Feb 1 20:28:09 CET 2011


On date Tuesday 2011-02-01 17:32:48 +0100, Michael Niedermayer encoded:
> On Sun, Jan 30, 2011 at 07:15:29PM +0100, Stefano Sabatini wrote:
[...]
> > -    if(vfilters) {
> > +    if (vfilters) {
> >          AVFilterInOut *outputs = av_malloc(sizeof(AVFilterInOut));
> >          AVFilterInOut *inputs  = av_malloc(sizeof(AVFilterInOut));
> >  
> 
> I dont want to nitpick but this has nothing to do with factorizing the code
> 
> Also vgraph was a local variable before and can stay one, it doesnt need to be
> moved to VideoState
> AVFilterGraph *graph = avfilter_graph_alloc();
> can stay outside configure_video_filters(graph)

Fixed&updated.
-- 
FFmpeg = Fucking and Fantastic Martial Plastic Elected Geek



More information about the ffmpeg-devel mailing list