[FFmpeg-devel] [PATCH] avcodec/hevc_sei: Support HEVC paired fields.

Brian Matherly brian.matherly at yahoo.com
Fri Sep 22 00:57:28 EEST 2017


On 9/14/2017 8:39 PM, Brian Matherly wrote:
> Correctly set frame.interlaced and frame.top_field_first when pic_struct
> indicates paired fields.
> ---
>   libavcodec/hevc_sei.c | 4 ++--
>   1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/libavcodec/hevc_sei.c b/libavcodec/hevc_sei.c
> index cd55d50..d0f9966 100644
> --- a/libavcodec/hevc_sei.c
> +++ b/libavcodec/hevc_sei.c
> @@ -137,10 +137,10 @@ static int decode_nal_sei_pic_timing(HEVCSEIContext *s, GetBitContext *gb, const
>       if (sps->vui.frame_field_info_present_flag) {
>           int pic_struct = get_bits(gb, 4);
>           h->picture_struct = AV_PICTURE_STRUCTURE_UNKNOWN;
> -        if (pic_struct == 2) {
> +        if (pic_struct == 2 || pic_struct == 10 || pic_struct == 12) {
>               av_log(logctx, AV_LOG_DEBUG, "BOTTOM Field\n");
>               h->picture_struct = AV_PICTURE_STRUCTURE_BOTTOM_FIELD;
> -        } else if (pic_struct == 1) {
> +        } else if (pic_struct == 1 || pic_struct == 9 || pic_struct == 11) {
>               av_log(logctx, AV_LOG_DEBUG, "TOP Field\n");
>               h->picture_struct = AV_PICTURE_STRUCTURE_TOP_FIELD;
>           }

Ping.

I think this should be an obvious change. If you look at table D.2 in 
Rec. ITU-T H.265, these four values are for fields.

~Brian

.


More information about the ffmpeg-devel mailing list