[FFmpeg-devel] [PATCH 1/3] g729dec: Add ff_ prefix to g729_postfilter() and g729_adaptive_gain_control()

Michael Niedermayer michaelni at gmx.at
Sun Sep 25 22:38:36 CEST 2011


On Sun, Sep 25, 2011 at 10:09:44PM +0200, Vitor Sessak wrote:
> On Sun, Sep 25, 2011 at 10:02 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> > On Sun, Sep 25, 2011 at 09:47:25PM +0200, Vitor Sessak wrote:
> >> On Sun, Sep 25, 2011 at 9:13 PM, Michael Niedermayer <michaelni at gmx.at> wrote:
> >> > On Sun, Sep 25, 2011 at 08:07:32PM +0200, Vitor Sessak wrote:
> >> >> ---
> >> >>  libavcodec/g729dec.c        |    4 ++--
> >> >>  libavcodec/g729postfilter.c |    4 ++--
> >> >>  libavcodec/g729postfilter.h |    4 ++--
> >> >>  3 files changed, 6 insertions(+), 6 deletions(-)
> >> >
> >> > applied, thanks
> >>
> >> Thanks. BTW, since I suppose you have already put everything in place
> >> to test this decoder, can't you make a FATE test out of the
> >> conformance vectors?
> >
> > This may be tricky due to floating pointg useage in the code
> 
> I don't see any FP usage in the code, can you elaborate?

I thought i saw a log() or exp() in there somewhere
seems this must have been an older version of the code

so consider that comment withdrawn


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Many things microsoft did are stupid, but not doing something just because
microsoft did it is even more stupid. If everything ms did were stupid they
would be bankrupt already.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20110925/a7f95ae7/attachment.asc>


More information about the ffmpeg-devel mailing list