On 4/5/15, Michael Niedermayer <michae...@gmx.at> wrote: > On Sat, Apr 04, 2015 at 10:33:44AM +0000, Paul B Mahol wrote: > [...] >> diff --git a/libavfilter/vf_cropdetect.c b/libavfilter/vf_cropdetect.c >> index e8492cd..485ae69 100644 >> --- a/libavfilter/vf_cropdetect.c >> +++ b/libavfilter/vf_cropdetect.c >> @@ -62,8 +62,10 @@ static int query_formats(AVFilterContext *ctx) >> AV_PIX_FMT_NONE >> }; >> >> - ff_set_common_formats(ctx, ff_make_format_list(pix_fmts)); >> - return 0; >> + AVFilterFormats *fmts_list = ff_make_format_list(pix_fmts); > > mixed declarations and code
I do not see it here. > > > [...] >> diff --git a/libavfilter/vf_histogram.c b/libavfilter/vf_histogram.c >> index 40ec400..a4a7f8c 100644 >> --- a/libavfilter/vf_histogram.c >> +++ b/libavfilter/vf_histogram.c >> @@ -121,9 +121,10 @@ static int query_formats(AVFilterContext *ctx) >> av_assert0(0); >> } >> >> - ff_set_common_formats(ctx, ff_make_format_list(pix_fmts)); >> - >> - return 0; >> + AVFilterFormats *fmts_list = ff_make_format_list(pix_fmts); > > mixed declarations and code Fixed. > > rest probably ok > > [...] > -- > Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB > > DNS cache poisoning attacks, popular search engine, Google internet > authority > dont be evil, please > _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel