diff mbox series

[FFmpeg-devel,major,bump,3/6] avcodec/dynamic_hdr_vivid: fix base_param_Delta

Message ID tencent_2F000D6AEE7C0AAE8E3B01C1B9B53C6B660A@qq.com
State New
Headers show
Series [FFmpeg-devel,major,bump,1/6] libavutil/hdr_dynamic_vivid_metadata: fix AVHDRVividColorToneMappingParams | expand

Checks

Context Check Description
yinshiyou/make_loongarch64 success Make finished
yinshiyou/make_fate_loongarch64 success Make fate finished
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished

Commit Message

Zhao Zhili Feb. 2, 2023, 7:02 a.m. UTC 
From: Zhao Zhili <zhilizhao@tencent.com>

It conflicts the comments. The operation based on Delta_enable_mode
can be applied by user during tone mapping.

Signed-off-by: Zhao Zhili <zhilizhao@tencent.com>
---
 libavcodec/dynamic_hdr_vivid.c | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)
diff mbox series

Patch

diff --git a/libavcodec/dynamic_hdr_vivid.c b/libavcodec/dynamic_hdr_vivid.c
index 4bc1cd3c70..a5ac0ce0c4 100644
--- a/libavcodec/dynamic_hdr_vivid.c
+++ b/libavcodec/dynamic_hdr_vivid.c
@@ -90,10 +90,7 @@  int ff_parse_itu_t_t35_to_dynamic_hdr_vivid(AVDynamicHDRVivid *s, const uint8_t
                         tm_params->base_param_k2 = get_bits(gb, 2);
                         tm_params->base_param_k3 = get_bits(gb, 4);
                         tm_params->base_param_Delta_enable_mode = get_bits(gb, 3);
-                        if (tm_params->base_param_Delta_enable_mode == 2 || tm_params->base_param_Delta_enable_mode == 6)
-                            tm_params->base_param_Delta = (AVRational){get_bits(gb, 7) * -1, base_param_Delta_den};
-                        else
-                            tm_params->base_param_Delta = (AVRational){get_bits(gb, 7), base_param_Delta_den};
+                        tm_params->base_param_Delta = (AVRational){get_bits(gb, 7), base_param_Delta_den};
 
                         if (get_bits_left(gb) < 1)
                             return AVERROR_INVALIDDATA;