]> git.sesse.net Git - ffmpeg/commit
lavc/qsvenc_h264: remove the privite option trellis
authorZhong Li <zhong.li@intel.com>
Wed, 27 Mar 2019 11:39:41 +0000 (19:39 +0800)
committerZhong Li <zhong.li@intel.com>
Thu, 28 Mar 2019 13:52:51 +0000 (21:52 +0800)
commit391f884675f319b95f5a72a410178516e11c557d
treed7fc7c87e82b4f27aa052aff04df386e11a9307b
parentb9a066ae23cbf488faf332cbfd101a3af5b77611
lavc/qsvenc_h264: remove the privite option trellis

There are many problems of current qsv trellis option:
1. Duplicated with AVCodecContext definition
2. MFX_TRELLIS_XXX is introduced by MSDK API 1.17
   Currently Without MSDK API checking thus may cause compilation issue.
3. user is inclined to enable trellis when set "-trellis 1", but
   actually it is to disable since MFX_TRELLIS_OFF is equal to 1.
4. It is too complex for user to enable trellis for every frame(I/P/B).

Just simply remove the private option, and switch to the AVCodecContext
definition. Compatibility should not a big problem (except can't exact map)
since the option name is same as AVCodecContext.

Signed-off-by: Zhong Li <zhong.li@intel.com>
Reviewed-by: Carl Eugen Hoyos <ceffmpeg@gmail.com>
Reviewed-by: Moritz Barsnick <barsnick@gmx.net>
libavcodec/qsvenc.h
libavcodec/qsvenc_h264.c
libavcodec/version.h