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

Michael Niedermayer michaelni at gmx.at
Mon Jul 6 02:26:16 CEST 2015

On Sat, Jul 04, 2015 at 05:00:39PM +0300, Ivan Uskov wrote:
> 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?

actually i think its ok

the CODEC_FLAG_INTERLACED_DCT flag is intended for the user to be
able to choose the CODING mode, that is it allows (not forced)
interlaced DCT CODING
the flag says nothing about the content being interlaced.

allowing interlaced coding just by common sense implies that the
material is probably interlaced

Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

No snowflake in an avalanche ever feels responsible. -- Voltaire
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20150706/11eaf496/attachment.sig>

More information about the ffmpeg-devel mailing list