[XviD-devel] [BUG] sad32v

[syskin at ihug.com.au]

Hi,

Edouard wrote:
> -        sad[1] = sad16(cur + 8, ref + 8, stride, 256*4096);
> -        sad[2] = sad16(cur + 8*stride, ref + 8*stride, stride, 256*4096);
> -        sad[3] = sad16(cur + 8*stride + 8, ref + 8*stride + 8, stride, 256*4096);
> +        sad[1] = sad16(cur + 16, ref + 16, stride, 256*4096);
> +        sad[2] = sad16(cur + 16*stride, ref + 16*stride, stride, 256*4096);
> +        sad[3] = sad16(cur + 16*stride + 16, ref + 16*stride + 16, stride, 256*4096);

Uh, of course. Isn't that the case? Does it actually say +8 ?? (I can't check atm, I'm at
uni).

If it's +8 in the code, then:
- it's the most stupid copy-and-paste bug ever
- it would explain why P/B/I decision gives so unexpected results from time to time.

Regards,
Radek