Re: [FFmpeg-devel] [PATCH v2] libavfilter/vf_blackdetect.c

2017-12-10 Thread Jonathan Baecker
Am 10.12.2017 um 13:48 schrieb Carl Eugen Hoyos: 2017-12-09 19:28 GMT+01:00 Jonathan Baecker : -static void check_black_end(AVFilterContext *ctx) +static void check_black(AVFilterContext *ctx) Why do you rename this function? I rename them because before the function only

Re: [FFmpeg-devel] [PATCH v2] libavfilter/vf_blackdetect.c

2017-12-10 Thread Carl Eugen Hoyos
2017-12-09 19:28 GMT+01:00 Jonathan Baecker : > -static void check_black_end(AVFilterContext *ctx) > +static void check_black(AVFilterContext *ctx) Why do you rename this function? Please remove the superfluous brackets in the "if()" you change. Carl Eugen