[Ffmpeg-devel] [PATCH] Update links to AMR reference source code in amr.c

Diego Biurrun diego
Fri Mar 30 08:34:26 CEST 2007


On Tue, Mar 27, 2007 at 10:34:54PM +0100, Robert Swain wrote:
> 
> On 26 Mar 2007, at 23:24, Michael Niedermayer wrote:
> >On Mon, Mar 26, 2007 at 09:52:27PM +0100, Robert Swain wrote:
> >>As per the subject, see attached. I don't know if older versions
> >>work, but the current floating point ones do.
> >
> >and the fixed point ones? whatever is updated should be tested before
> >i do remember some versions didnt work, thats why we dont link to
> >"latest" but rather specific revissions
> 
> The current fixed point version has some strange platform/arch stuff  
> in typedefs.h that doesn't include code for OS X/Intel. I've tried  
> working around it but it doesn't seem to build here because of the  
> definition of 'round'. But the floating point versions do work, I'll  
> send a patch for just those and updating the html instructions too if  
> desirable.

I've applied the relevant parts of your patch.  Try testing the
fixed-point version on an x86 box and report back.

Diego




More information about the ffmpeg-devel mailing list