On Wed, Mar 28, 2018 at 04:50:10PM +0800, Zhong Li wrote:
> Currently pict_type are unset.
> Add an extra param to fetch the picture type from qsv decoder
>
> v2: fix the compile error since AV_PICTURE_TYPE_NONE is not existed in libav.
> v3: remove the key_frame setting because the judgement “key frame is equal
> to IDR frame” only suitable for H264.
These look like sensible patch annotations that you can add with --annotate
when using git-send-email, but they should not be part of the log message.
How you fixed compilation failures needs not be preserved for posterity ;)
> --- a/libavcodec/qsv.c
> +++ b/libavcodec/qsv.c
> @@ -195,6 +195,30 @@ int ff_qsv_find_surface_idx(QSVFramesContext *ctx,
> QSVFrame *frame)
>
> +enum AVPictureType ff_qsv_map_pictype(int mfx_pic_type)
> +{
> + enum AVPictureType type;
> + switch (mfx_pic_type & 0x7) {
> + case MFX_FRAMETYPE_I:
Please indent switch and case at the same level.
Diego
_______________________________________________
libav-devel mailing list
[email protected]
https://lists.libav.org/mailman/listinfo/libav-devel
