[FFmpeg-devel] [PATCH 3/3] tests: Add fate-hevc-color-reserved

[Andreas Rheinhardt]

Zhao Zhili:
> 
> 
>> On May 21, 2025, at 21:28, Zhao Zhili <quinkblack-at-foxmail.com at ffmpeg.org> wrote:
>>
>>> On May 21, 2025, at 21:11, Andreas Rheinhardt <andreas.rheinhardt at outlook.com> wrote:
>>>
>>> Zhao Zhili:
>>>> From: Zhao Zhili <zhilizhao at tencent.com>
>>>>
>>>> ---
>>>> tests/fate/hevc.mak                | 3 +++
>>>> tests/ref/fate/hevc-color-reserved | 6 ++++++
>>>> 2 files changed, 9 insertions(+)
>>>> create mode 100644 tests/ref/fate/hevc-color-reserved
>>>>
>>>> diff --git a/tests/fate/hevc.mak b/tests/fate/hevc.mak
>>>> index 390ccf46e2..5e721526d0 100644
>>>> --- a/tests/fate/hevc.mak
>>>> +++ b/tests/fate/hevc.mak
>>>> @@ -294,6 +294,9 @@ FATE_HEVC-$(call FRAMECRC, MOV, HEVC) += fate-hevc-mv-position
>>>> fate-hevc-alpha: CMD = framecrc -i $(TARGET_SAMPLES)/hevc/alpha.mp4
>>>> FATE_HEVC-$(call FRAMECRC, MOV, HEVC) += fate-hevc-alpha
>>>>
>>>> +fate-hevc-color-reserved: CMD = framecrc -i $(TARGET_SAMPLES)/hevc/color_prim_reserved0.hevc -fps_mode passthrough -sws_flags +accurate_rnd+bitexact -vf scale,format=nv12
>>>> +FATE_HEVC-$(call FRAMECRC, HEVC, HEVC, SCALE_FILTER) += fate-hevc-color-reserved
>>>
>>> A new sample for this? Why don't you just create one with hevc_metadata?
> 
> On the other hand, we can create invalid sample with hevc_metadata for now.
> 
> Does it make sense to check primaries/trc/matrix in hevc_metadata?
> 

IMO reserved != invalid

- Andreas