In trying some changes I noticed the following 2 bugs
in the me_utils file.
This is for the none MMX case, the MMX case appears to
generate different files so I am assuming there are
other issues in that.
void BlockDiffQuarterPel::Diff( const BlockDiffParams&
const MVector& mv ,
const float mvcost,
const float lambda,
the bound check case the x, y params are not correct
and should be as in the previous QuarterPel Diff
routine for bounds checking, uses m_pic_data[y][x] but
x,y are using dparams.Xl() instead of dparams.Xp().
also in BiBlockQuarterPel::Diff
the second reference uses ref_start1 instead of
ref_start2 in the bounds_check case.
I'll try and produce a patch for this at some point
over the weekend.
Log in to post a comment.