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

[Brian Matherly]

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.com at 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