[FFmpeg-devel] [PATCH v2 1/6] avutil/csp: fix documentation of av_csp_trc_function

[Niklas Haas]

On Sat, 30 Nov 2024 15:23:46 +0100 Niklas Haas <ffmpeg at haasn.xyz> wrote:
> From: Niklas Haas <git at haasn.dev>
>
> This explanation was inaccurate and highly misleading. The new wording is taken
> more or less directly from ITU-T H.273, and also matches my understanding of
> these functions.
> ---
>  libavutil/csp.h | 8 ++++++--
>  1 file changed, 6 insertions(+), 2 deletions(-)
>
> diff --git a/libavutil/csp.h b/libavutil/csp.h
> index 73bce52bc0..bc8f3dc9f6 100644
> --- a/libavutil/csp.h
> +++ b/libavutil/csp.h
> @@ -81,8 +81,12 @@ typedef struct AVColorPrimariesDesc {
>  } AVColorPrimariesDesc;
>
>  /**
> - * Function pointer representing a double -> double transfer function that performs
> - * an EOTF transfer inversion. This function outputs linear light.
> + * Function pointer representing a double -> double transfer function that
> + * performs either an OETF transfer function, or alternatively an inverse EOTF
> + * function (in particular, for SMPTE ST 2084 / PQ). This function inputs
> + * linear light, and outputs gamma encoded light.
> + *
> + * See ITU-T H.273 for more information.
>   */
>  typedef double (*av_csp_trc_function)(double);
>
> --
> 2.47.0
>

Will merge this series (with patch "avutil/csp: eliminate redundant branch"
dropped) in a day or two if there are no objections.