[XviD-devel] ME bug

[Bryan Mayland]

Edouard Gomez wrote:

>at the end of diamond search stops, advDiamond search again
>neighboring pels
>but case 2 + 8 and case 1 + 8 shoud change to each other.
>  
>
He's right.  The switch code in xvid_me_AdvDiamondSearch() for 2 + 8 is 
checking for a Left + Down, and the 1 + 8 is checking for a Right + 
Down.  The code for the two case statements should be swapped.  I've 
checked the rest of them (R, L, R+U, U, D, L+U) and they appear to be 
correct.

I'm not sure what he's saying about "at end of diamond search stops" though.