On Jun 16, 2008, at 7:57 PM, Alexander Strange wrote: > --- > + switch (s->mv_type) { > + case MV_TYPE_16X16: > + off = (s->mv[dir][0][1] + 16) >> 5; > + break; > + default: > + goto unhandled; > + } Now that I look at it again this rounding seems wrong. I'll have to check that out.