[FFmpeg-devel] [PATCH] Electronic Arts TQI decoder

Michael Niedermayer michaelni
Sat Jan 31 15:37:12 CET 2009


On Sat, Jan 31, 2009 at 02:45:24PM +1100, Peter Ross wrote:
> On Fri, Jan 30, 2009 at 08:01:52PM +0100, Michael Niedermayer wrote:
> > On Sat, Jan 31, 2009 at 02:40:30AM +1100, Peter Ross wrote:
> > > On Mon, Jan 26, 2009 at 11:17:57AM +0100, Michael Niedermayer wrote:
> > > > On Sun, Jan 25, 2009 at 11:06:25PM +1100, Peter Ross wrote:
> > > > > Patch enclosed.
> > > 
> > > Updated.
> 
> Updated.
> 
> > [...]
> > > +static void tqi_calculate_qtable(MpegEncContext *s, int quant)
> > > +{
> > > +    const double flt_value = (107.5-quant)*0.625;
> > > +    int i;
> > > +    s->intra_matrix[0] = (ff_inv_aanscales[0]*ff_mpeg1_default_intra_matrix[0])>>11;
> > > +    for(i=1; i<64; i++)
> > > +        s->intra_matrix[i] = (int)(ff_inv_aanscales[i]*ff_mpeg1_default_intra_matrix[i]*flt_value + 2)>>10;
> > > +}
> > 
> > i suspect that the ff_inv_aanscales is wrong for the normal IDCTs
> > the normal ones should have a "flat" scaling
> 
> Yes, of course.

looks ok

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

When you are offended at any man's fault, turn to yourself and study your
own failings. Then you will forget your anger. -- Epictetus
-------------- 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/20090131/831c28a0/attachment.pgp>



More information about the ffmpeg-devel mailing list