Hi,
2015-10-01 22:34 GMT+02:00 Hendrik Leppkes <h.lepp...@gmail.com>:
>> + avctx->coded_width = FFALIGN(avctx->width, 16);
>> + avctx->coded_height = FFALIGN(avctx->coded_height, 16);
>
> Intentional that this is not ->height, but ->coded_height?
Huh, yes, and the worse is it was intended to fix coded_height.
Here's an updated patch.
--
Christophe
From c10e86ce4469b81ac66b7fa2dd233efd191f508c Mon Sep 17 00:00:00 2001
From: Christophe Gisquet <christophe.gisq...@gmail.com>
Date: Wed, 30 Sep 2015 10:14:59 +0200
Subject: [PATCH 1/3] dnxhddec: initialize with mb-aligned dimensions
The coded size is a multiple of the macroblock size, which is 16.
---
libavcodec/dnxhddec.c | 3 +++
1 file changed, 3 insertions(+)
diff --git a/libavcodec/dnxhddec.c b/libavcodec/dnxhddec.c
index e3dec78..755cf9a 100644
--- a/libavcodec/dnxhddec.c
+++ b/libavcodec/dnxhddec.c
@@ -87,6 +87,9 @@ static av_cold int dnxhd_decode_init(AVCodecContext *avctx)
ctx->cid = -1;
avctx->colorspace = AVCOL_SPC_BT709;
+ avctx->coded_width = FFALIGN(avctx->width, 16);
+ avctx->coded_height = FFALIGN(avctx->height, 16);
+
ctx->rows = av_mallocz_array(avctx->thread_count, sizeof(RowContext));
if (!ctx->rows)
return AVERROR(ENOMEM);
--
2.5.2
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
