[FFmpeg-devel] [PATCH] avcodec/hevcdec: Fix precedence, bogus film grain warning

Andreas Rheinhardt andreas.rheinhardt at outlook.com
Fri Apr 19 18:53:28 EEST 2024


Andreas Rheinhardt:
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
> ---
>  libavcodec/hevcdec.c | 8 ++++----
>  1 file changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/libavcodec/hevcdec.c b/libavcodec/hevcdec.c
> index a2c967cf09..1974bf9cb3 100644
> --- a/libavcodec/hevcdec.c
> +++ b/libavcodec/hevcdec.c
> @@ -2898,10 +2898,10 @@ static int hevc_frame_start(HEVCContext *s)
>          goto fail;
>  
>      if (s->ref->needs_fg &&
> -        ( s->sei.common.film_grain_characteristics.present &&
> -          !ff_h274_film_grain_params_supported(s->sei.common.film_grain_characteristics.model_id,
> -                                             s->ref->frame->format))
> -          || !av_film_grain_params_select(s->ref->frame)) {
> +        (s->sei.common.film_grain_characteristics.present &&
> +         !ff_h274_film_grain_params_supported(s->sei.common.film_grain_characteristics.model_id,
> +                                              s->ref->frame->format)
> +         || !av_film_grain_params_select(s->ref->frame))) {
>          av_log_once(s->avctx, AV_LOG_WARNING, AV_LOG_DEBUG, &s->film_grain_warning_shown,
>                      "Unsupported film grain parameters. Ignoring film grain.\n");
>          s->ref->needs_fg = 0;

Approved by Niklas on IRC and pushed.

- Andreas



More information about the ffmpeg-devel mailing list