Re: [FFmpeg-devel] [PATCH 1/3] dnxhddec: initialize with mb-aligned dimensions

2015-10-02 Thread Michael Niedermayer
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

Re: [FFmpeg-devel] [PATCH 1/3] dnxhddec: initialize with mb-aligned dimensions

2015-10-01 Thread Christophe Gisquet
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.

Re: [FFmpeg-devel] [PATCH 1/3] dnxhddec: initialize with mb-aligned dimensions

2015-10-01 Thread Hendrik Leppkes
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 >

[FFmpeg-devel] [PATCH 1/3] dnxhddec: initialize with mb-aligned dimensions

2015-10-01 Thread Christophe Gisquet
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