[FFmpeg-devel] [PATCH] Fix PSX MDEC decoder

Michael Niedermayer michaelni
Sun Jan 2 04:43:07 CET 2011


On Sat, Jan 01, 2011 at 07:52:38PM +0100, Vitor Sessak wrote:
> Hi,
>
> I saw in a blog post [1] that FFmpeg decodes Playstation 1 video files  
> with artifacts. It is indeed true, due to a wrong pixfmt (YUV/YUVJ) and  
> not imposing using DCT_SIMPLE, as is done in MPEG1. The attached patch  
> fixes it.
>
> For reference, I'm attaching a decoded frame with:
>
> 1- Current SVN with -idct mmx
> 2- Current SVN with -idct simple
> 3- Patched SVN
> 4- Reference decoder output (i.e., PS1 image grab)
>
> -Vitor
>
> [1]  
> http://jpsxdec.blogspot.com/2010/09/psx-video-decoders-final-showdown.html
>

>  mdec.c |    4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 5c901ccdc3ae44b4a0fc43246c15ee462603a563  fix_mdec.diff

if this fixes a bug, ok of course
dont forget testing /updating fate


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

The worst form of inequality is to try to make unequal things equal.
-- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/ffmpeg-devel/attachments/20110102/df9bed90/attachment.pgp>



More information about the ffmpeg-devel mailing list