[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