2018-05-23 0:21 GMT+02:00, Hendrik Leppkes <[email protected]>: > On Tue, May 22, 2018 at 10:35 PM, Carl Eugen Hoyos <[email protected]> > wrote: >> 2018-05-22 17:40 GMT+02:00, Gagandeep Singh <[email protected]>: >> >>> + 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. >> > > Except, no sane developer works like that.
It's great to know that you are so much smarter than most developers, I will forward the next regression directly to you to fix it. Thank you, Carl Eugen _______________________________________________ ffmpeg-devel mailing list [email protected] http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
