[FFmpeg-soc] [soc]: r766 - qcelp/qcelpdec.c

Aurelien Jacobs aurel at gnuage.org
Tue Aug 14 19:13:06 CEST 2007


On Tue, 14 Aug 2007 12:24:22 -0400
"Reynaldo Verdejo" <rverdejo at gmail.com> wrote:

> Hi Aurelien
> 
> On 8/14/07, Aurelien Jacobs <aurel at gnuage.org> wrote:
> 
> > > -        outbuffer[i]=av_clip(lrintf(4*ppf_vector[i]), -32768, 32767);
> > > +        outbuffer[i]=av_clip(lrintf(4*ppf_vector[i]), -32768, 32768);
> >
> > This looks strange. The maximum 16bits signed integer is 32767, not 32768.
> 
> 
> Fixed, thanks.
> 
> And if the range you want is indeed -32768 - 32767, you'd better use
> > av_clip_int16().
> 
> 
> It would be nice but I can't find it...

You need a very recent SVN checkout of ffmpeg.

Aurel



More information about the FFmpeg-soc mailing list