On 9/26/2017 8:38 PM, Michael Niedermayer wrote:
On Sun, Sep 24, 2017 at 10:13:11PM -0500, Brian Matherly wrote:
From: Brian Matherly <brian.matherly-at-yahoo....@ffmpeg.org>

Correctly set the interlaced_frame and top_field_first fields when pic_struct
indicates paired fields.
---
  libavcodec/hevc_sei.c             |   4 +-
  tests/fate/hevc.mak               |   6 +-
  tests/ref/fate/hevc-paired-fields | 120 ++++++++++++++++++++++++++++++++++++++
  3 files changed, 127 insertions(+), 3 deletions(-)
  create mode 100644 tests/ref/fate/hevc-paired-fields
the test seems to fail on big endian: (mips qemu)

@@ -14,7 +14,7 @@
  pkt_size=229528
  width=1920
  height=540
-pix_fmt=yuv422p10le
+pix_fmt=yuv422p10be
  sample_aspect_ratio=1:1
  pict_type=I
  coded_picture_number=0
@@ -44,7 +44,7 @@
  pkt_size=95954
  width=1920
  height=540
-pix_fmt=yuv422p10le
+pix_fmt=yuv422p10be
  sample_aspect_ratio=1:1
  pict_type=B
  coded_picture_number=0
@@ -74,7 +74,7 @@
  pkt_size=114837
  width=1920
  height=540
-pix_fmt=yuv422p10le
+pix_fmt=yuv422p10be
  sample_aspect_ratio=1:1
  pict_type=B
  coded_picture_number=0
@@ -104,7 +104,7 @@
  pkt_size=85098
  width=1920
  height=540
-pix_fmt=yuv422p10le
+pix_fmt=yuv422p10be
  sample_aspect_ratio=1:1
  pict_type=B
  coded_picture_number=0

[...]

The pix_fmt is not consequential to the test. In fact, most of the fields are not related to the test. Patch V4 reduces the test to only the fields relevant to the paired fields feature of the sample.

Thanks,

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

Reply via email to