From b8ac69eef184006a8abee89befc95f88e9da3f30 Mon Sep 17 00:00:00 2001 From: Pekka Paalanen Date: Tue, 7 Oct 2025 14:45:38 +0300 Subject: [PATCH] color: check maxCLL and maxFALL against target lum always The target luminance range is implied when not explicitly set, so maxCLL and maxFALL should be checked against target luminances regardless of whether target luminances are explicit. Signed-off-by: Pekka Paalanen --- libweston/color-profile-param-builder.c | 6 ------ 1 file changed, 6 deletions(-) diff --git a/libweston/color-profile-param-builder.c b/libweston/color-profile-param-builder.c index d86c8ed6d..db3c06f27 100644 --- a/libweston/color-profile-param-builder.c +++ b/libweston/color-profile-param-builder.c @@ -758,9 +758,6 @@ validate_maxcll(struct weston_color_profile_param_builder *builder) maxCLL); } - if (!(builder->group_mask & WESTON_COLOR_PROFILE_PARAMS_TARGET_LUMINANCE)) - return; - if (builder->params.target_min_luminance >= maxCLL) store_error(builder, WESTON_COLOR_PROFILE_PARAM_BUILDER_ERROR_INVALID_LUMINANCE, "maxCLL (%f) should be greater than target min luminance (%f)", @@ -783,9 +780,6 @@ validate_maxfall(struct weston_color_profile_param_builder *builder) maxFALL); } - if (!(builder->group_mask & WESTON_COLOR_PROFILE_PARAMS_TARGET_LUMINANCE)) - return; - if (builder->params.target_min_luminance >= maxFALL) store_error(builder, WESTON_COLOR_PROFILE_PARAM_BUILDER_ERROR_INVALID_LUMINANCE, "maxFALL (%f) must be greater than min luminance (%f)",