[FFmpeg-devel] [PATCH][ticket #5522] lavc/cfhd: interlaced frame decoding added

Carl Eugen Hoyos ceffmpeg at gmail.com
Tue May 22 23:35:12 EEST 2018


2018-05-22 17:40 GMT+02:00, Gagandeep Singh <deepgagan231197 at gmail.com>:

> +            low    = s->plane[plane].subband[0];
> +            high   = s->plane[plane].subband[8];
> +            output = s->plane[plane].l_h[6];
> +            for (i = 0; i < lowpass_width; i++) {
> +                vert_filter(output, lowpass_width, low, lowpass_width,
> high, highpass_stride, lowpass_height);
> +                low++;
> +                high++;
> +                output++;
> +            }
>
> -        low    = s->plane[plane].subband[0];
> -        high   = s->plane[plane].subband[8];
> -        output = s->plane[plane].l_h[6];
> -        for (i = 0; i < lowpass_width; i++) {
> -            vert_filter(output, lowpass_width, low, lowpass_width, high,
> highpass_stride, lowpass_height);
> -            low++;
> -            high++;
> -            output++;
> -        }

The patch will get much more readable (and easier to review)
if you do not re-indent, instead send a second patch with the
cosmetic changes only.

Carl Eugen


More information about the ffmpeg-devel mailing list