On Tue, 15 Jan 2019 at 11:57, Carl Eugen Hoyos <ceffm...@gmail.com> wrote:

> 2019-01-15 11:30 GMT+01:00, Rostislav Pehlivanov <atomnu...@gmail.com>:
> > On Tue, 15 Jan 2019 at 09:24, Shaofei Wang <shaofei.w...@intel.com>
> wrote:
>
> >> +#if HAVE_THREADS
> >> +        if (!abr_pipeline) {
> >> +            ret = reap_filters(1);
> >> +        } else {
> >> +            ret = pipeline_reap_filters(1, ifilter);
> >> +        }
> >>
> >
> > Same.
>
> This hunk should have brackets, it simplifies
> debugging and future patches at very little cost.
>

No, it does not. It wastes a line.


> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel@ffmpeg.org
> http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to