[Theora-dev] Re: MMX/mmxext optimisations

[Adam D. Moss]

Wim Taymans wrote:
> oh, right. GetHalfPixelSumAbsDiffs in the reference encoder has a bug
> where ErrorSoFar is added twice to DiffVall. That would also make a
> difference. Error like this:
> 
> --- mcomp.c     2003-12-03 09:59:41.000000000 +0100
> +++ mcomp.c.fix 2004-08-26 15:40:48.000000000 +0200
> @@ -272,7 +272,7 @@ static ogg_uint32_t GetHalfPixelSumAbsDi
> 
>    if ( RefOffset == 0 ) {
>      /* Simple case as for non 0.5 pixel */
> -    DiffVal += GetSumAbsDiffs( SrcData, RefDataPtr1, PixelsPerLine,
> +    DiffVal = GetSumAbsDiffs( SrcData, RefDataPtr1, PixelsPerLine,
>                                 ErrorSoFar);
>    } else  {
>      for ( i=0; i < BLOCK_HEIGHT_WIDTH; i++ ) {
> 

*bump*

I just looked into this and I think Wim's right.  Could this
patch be evaluated and committed to the trunk...?

Regards,
--Adam