[FFmpeg-devel] [PATCH] libavcodec/qsvenc.c: More correct selection of alignment of a frame height depending whether an encoded sequence progressive or not.

[Ivan Uskov]

Hello Hendrik,

The harm of hard-coded alignment 32 that typical 1280x720 progressive
video should be encoded as 1280x736. I.e. absolutely superfluous
stripe of macroblocks should be encoded.
I can see that CODEC_FLAG_INTERLACED_DCT currently checks into
dnxhddata.c
dnxhdenc.c
dvenc.c
libx264.c
libxavs.c
mpegvideo_enc.c
nvenc.c
Why it can not be used by similar way in qsvenc.c?



Saturday, July 4, 2015, 4:34:32 PM, you wrote:

HL> On Sat, Jul 4, 2015 at 2:38 PM, Ivan Uskov <ivan.uskov at nablet.com> wrote:
>> Hello All,
>>
>> There is patch which fixes too strong frame height rounding for case
>> when progressive frame sequence encodes by QSV.

HL> Is there any harm in always using 32 alignment for the height?
HL> Because the flag you are checking there is not correct. In fact, there
HL> is no reliable global interlaced flag.


-- 
Best regards,
 Ivan                            mailto:ivan.uskov at nablet.com