On Thu, Oct 01, 2015 at 11:11:16PM +0200, Christophe Gisquet wrote:
> Hi,
>
> 2015-10-01 22:34 GMT+02:00 Hendrik Leppkes :
> >> +avctx->coded_width = FFALIGN(avctx->width, 16);
> >> +avctx->coded_height = FFALIGN(avctx->coded_height, 16);
> >
> > Intentional that this is not ->height, but
Hi,
2015-10-01 22:34 GMT+02:00 Hendrik Leppkes :
>> +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.
On Thu, Oct 1, 2015 at 10:01 PM, Christophe Gisquet
wrote:
> 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..3985318 100644
>
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..3985318 100644
--- a/libavcodec/dnxhddec.c
+++ b/libavcodec/dnxhddec.c
@@ -87,6 +87,9 @@ s