[FFmpeg-devel] Re: [PATCH] libavfilter: mark comp_expr_str as const in LutContext

[Michael Niedermayer via ffmpeg-devel <ffmpeg-devel@ffmpeg.org>]

[-- Attachment #1.1: Type: text/plain, Size: 1472 bytes --]

Hi Adeba

On Wed, Jan 14, 2026 at 03:17:15PM +0530, Adeba via ffmpeg-devel wrote:
> Hello,
> 
> This patch marks comp_expr_str in LutContext as const to improve code
> safety.
> 
> Thanks,
> Adeba Shahnaz

>  vf_lut.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> ac8e5f6f1618b72ab39c74d98ec218365ec6224e  /var0001-PATCH-libavfilter-mark-comp_expr_str-as-const-in-Lut.patch
> From c8bcd080f602bab8cd4e399db528377010ea8197 Mon Sep 17 00:00:00 2001
> From: Adeeba Shahnaz <adeebashahnaz@Adeebas-MacBook-Pro.local>
> Date: Wed, 14 Jan 2026 15:15:23 +0530
> Subject: [PATCH] [PATCH] libavfilter: mark comp_expr_str as const in
>  LutContext
> 
> ---
>  libavfilter/vf_lut.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavfilter/vf_lut.c b/libavfilter/vf_lut.c
> index d15fb85c4e..3acb08780c 100644
> --- a/libavfilter/vf_lut.c
> +++ b/libavfilter/vf_lut.c
> @@ -64,7 +64,7 @@ enum var_name {
>  typedef struct LutContext {
>      const AVClass *class;
>      uint16_t lut[4][256 * 256];  ///< lookup table for each component
> -    char   *comp_expr_str[4];
> +    const  char  *comp_expr_str[4];
>      AVExpr *comp_expr[4];

why the number of spaces ?

thx

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

For a strong democracy, genuine criticism is necessary, allegations benefit
noone, they just cause unnecessary conflicts. - Narendra Modi

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 bytes --]

[-- Attachment #2: Type: text/plain, Size: 163 bytes --]

_______________________________________________
ffmpeg-devel mailing list -- ffmpeg-devel@ffmpeg.org
To unsubscribe send an email to ffmpeg-devel-leave@ffmpeg.org