[FFmpeg-devel] [PATCH] Fix partially hidden macroblocks for 10-bit DNxHD

[Michael Niedermayer]

On Thu, Aug 23, 2012 at 03:42:37PM +0100, Joseph Artsimovich wrote:
> Hello,
> 
> The attached patch fixes "mirroring" artifacts at the bottom of a
> frame when encoding 10-bit DNxHD in interlaced (-flags +ildct) mode.
> The problem was in dnxhd_10bit_get_pixels_8x4_sym(). In addition to
> producing the above mentioned "mirroring", it was also assuming
> sizeof(DCTELEM) == 2, which might be correct now, but should not be
> relied upon.
> The new version is very close to dnxhd_8bit_get_pixels_8x4_sym().
> 
> Progressive mode wasn't affected because 1080 and 720 are both
> multiples of 8,  meaning that partially hidden blocks would never
> happen.
> 
> -- 
> Joseph Artsimovich
> Senior C++ Applications Developer
> MirriAd Ltd
> 

>  dnxhdenc.c |   16 ++++++++++++----
>  1 file changed, 12 insertions(+), 4 deletions(-)
> 4e5e4a610c65b6eda9566dc7d9cad4ea341f5cea  0001-Fix-partially-hidden-macroblocks-for-10-bit-DNxHD.patch
> From 924c4a00935aa435e923587257e939366d926580 Mon Sep 17 00:00:00 2001
> From: Joseph Artsimovich <joseph at mirriad.com>
> Date: Thu, 23 Aug 2012 14:51:28 +0100
> Subject: [PATCH] Fix partially hidden macroblocks for 10-bit DNxHD.

bugfix confirmed

patch applied

thanks

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

The real ebay dictionary, page 3
"Rare item" - "Common item with rare defect or maybe just a lie"
"Professional" - "'Toy' made in china, not functional except as doorstop"
"Experts will know" - "The seller hopes you are not an expert"
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20120823/51011b2c/attachment.asc>