[FFmpeg-devel] [PATCH] EA TGQ video fixes

[Vitor Sessak]

Ramiro Polla wrote:
> On Wed, May 20, 2009 at 5:41 PM, Reimar D?ffinger
> <Reimar.Doeffinger at gmx.de> wrote:
>> On Wed, May 20, 2009 at 05:31:48PM -0300, Ramiro Polla wrote:
>>> On Wed, May 20, 2009 at 5:07 PM, Vitor Sessak <vitor1001 at gmail.com> wrote:
>>> [...]
>>>> Anyway, again, does anyone get a different output for:
>>>>
>>>> ffmpeg -idct simple -i
>>>> http://samples.mplayerhq.hu/game-formats/ea-tgq-uv/tgq/v27.tgq -an -t 10 -f
>>>> framecrc - | md5sum
>>>>
>>>> ?
>>>> I get 2b84588602acda70269bd3e3aa1ce632
>>> Wine gives:
>>> e4c063ac0f26e12d63c3e293066eef10
>> Have you time to test with the not yet applied patch?
> 
> The one that moves the block to context? It doesn't change anything.
> 
> I guess it should be reproducible with any wine, so your 1) condition
> on the previous message is satisfied =)

Thanks, I succeeded in reproducing it with wine (only using apt-get 
installed software on Ubuntu 8.04, btw, I only needed to hack configure 
to accept my version of mingw).

It looks like the problem is in ff_simple_idct_put(). The attached patch 
should expose the problem. In linux, I get

-- test --
  0 0 0 0 0 0 0 0
  0 255 255 255 0 0 0 0
  0 255 255 255 0 0 0 0
  0 255 255 255 0 0 0 0
  0 255 255 255 0 0 0 0
  0 255 255 255 0 0 0 0
  0 255 255 255 0 0 0 0
  0 255 255 255 0 0 0 0

and in wine

-- test --
  0 0 0 0 64 0 0 0
  0 255 255 255 0 0 0 0
  0 255 255 255 0 0 0 0
  0 255 255 255 0 0 0 0
  0 255 255 255 0 0 0 0
  0 255 255 255 0 0 0 0
  0 255 255 255 0 0 0 0
  0 255 255 255 0 0 0 0

-Vitor
-------------- next part --------------
A non-text attachment was scrubbed...
Name: ea_bug.diff
Type: text/x-diff
Size: 1437 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20090521/ad7b82d3/attachment.diff>