> -----Original Message-----
> From: Andreas Rheinhardt <andreas.rheinha...@outlook.com>
> Sent: Saturday, November 27, 2021 3:03 PM
> To: ffmpeg-devel@ffmpeg.org
> Cc: Andreas Rheinhardt <andreas.rheinha...@outlook.com>; Soft Works
> <softwo...@hotmail.com>
> Subject: [PATCH] avfilter/vf_paletteuse: Add missing parentheses
> 
> Fixes a mistake in dea673d0d548c864ec85f9260d8900d944ef7a2a.
> GCC emitted a -Wint-in-bool-context warning because of that.
> 
> Reviewed-by: Soft Works <softwo...@hotmail.com>
> Signed-off-by: Andreas Rheinhardt <andreas.rheinha...@outlook.com>
> ---
> Will apply this tonight unless there are objections.
> 
>  libavfilter/vf_paletteuse.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavfilter/vf_paletteuse.c b/libavfilter/vf_paletteuse.c
> index 5f20cd75b2..5b300be5a5 100644
> --- a/libavfilter/vf_paletteuse.c
> +++ b/libavfilter/vf_paletteuse.c
> @@ -811,7 +811,7 @@ static void debug_mean_error(PaletteUseContext *s, const
> AVFrame *in1,
>      uint8_t  *src2 =             in2->data[0];
>      const int src1_linesize = in1->linesize[0] >> 2;
>      const int src2_linesize = in2->linesize[0];
> -    const float div = in1->width * in1->height * s->use_alpha ? 4 : 3;
> +    const float div = in1->width * in1->height * (s->use_alpha ? 4 : 3);
>      unsigned mean_err = 0;
> 
>      for (y = 0; y < in1->height; y++) {
> --
> 2.32.0

LGTM

Thanks for spotting.

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to