[FFmpeg-devel] [PATCH] QCELP decoder

Michael Niedermayer michaelni
Thu Oct 16 18:32:10 CEST 2008


On Thu, Oct 16, 2008 at 05:34:28PM +0200, Diego Biurrun wrote:
> On Tue, Oct 14, 2008 at 06:59:34PM -0700, Kenan Gillet wrote:
> > 
> > Here is a summary of the round 4 patch:
[...]
> > +        gain[0] =     ga[0];
> > +        gain[1] = 0.6*ga[0]+0.4*ga[1];
> > +        gain[2] =     ga[1];
> > +        gain[3] = 0.2*ga[1]+0.8*ga[2];
> > +        gain[4] = 0.8*ga[2]+0.2*ga[3];
> > +        gain[5] =     ga[3];
> > +        gain[6] = 0.4*ga[3]+0.6*ga[4];
> > +        gain[7] =     ga[4];
> 
> A few spaces between operators would make this look less cramped.

yes, but only surrounding the + :)


[...]
> > +    .long_name = NULL_IF_CONFIG_SMALL("QCLEP / PureVoice"),
> 
> QCELP

Actually iam wondering if this shouldnt be the full title of the spec,
i suspect it has some number in it ... but then i did not check

[..]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

I am the wisest man alive, for I know one thing, and that is that I know
nothing. -- Socrates
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20081016/31240c5e/attachment.pgp>



More information about the ffmpeg-devel mailing list