On Wed, Jan 20, 2016 at 4:59 AM, Diego Biurrun <[email protected]> wrote:
>> +        src += 2 * src_wrap;
>> +        dst += dst_wrap;
>> +    }
>> +}
>
> You've reformatted the code while moving.  This is nice and all, but I fear
> it might throw off the Git copy detection routines.  I would suggest doing
> a reformatting as preliminary step instead.

doesn't git copy work only on full files anyway?

>> +/* 8x8 -> 1x1 */
>> +static void shrink88(uint8_t *dst, int dst_wrap,
>> +                     const uint8_t *src, int src_wrap,
>> +                     int width, int height)
>> +{
>> +    int w, i;
>> +
>> +    for (; height > 0; height--) {
>> +        for(w = width;w > 0; w--) {
>> +            int tmp = 0;
>> +            for (i = 0; i < 8; i++) {
>> +                tmp += src[0] + src[1] + src[2] + src[3] +
>> +                       src[4] + src[5] + src[6] + src[7];
>> +                src += src_wrap;
>> +            }
>> +            *(dst++) = (tmp + 32)>>6;
>
> spaces around >>

ok
-- 
Vittorio
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel

Reply via email to