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

[Peter Ross]

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.

-- Peter
(A907 E02F A6E5 0CD2 34CD 20D2 6760 79C5 AC40 DD6B)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: eatqi-decoder-r5.diff
Type: text/x-diff
Size: 8005 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090131/22f620ac/attachment.diff>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: mpeg12-global-intra-block-r2.diff
Type: text/x-diff
Size: 1824 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090131/22f620ac/attachment-0001.diff>
-------------- 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/22f620ac/attachment.pgp>