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

.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to