[FFmpeg-devel] [PATCH 0/2] Animated JPEG XL Encoder

Marth64 marth64 at proxyid.net
Thu Dec 12 00:32:07 EET 2024


Hi Leo,

In libjxl_anim_encode_frame():

> +        if (!ctx->prev) {
> +            ret = AVERROR(ENOMEM);
> +            goto end;
> +        }
> +        ret = ff_encode_get_frame(avctx, ctx->prev);
> +        if (ret == AVERROR(EAGAIN) || ret == AVERROR_EOF)
> +            goto end;
> +        ret = libjxl_preprocess_stream(avctx, ctx->prev, 1);
> +        if (ret < 0)
> +            goto end;

and

> +    if (ret == AVERROR(EAGAIN))
> +        goto end;

As `end` block simply returns ret, can these gotos be eliminated and
replaced with a return on the spot?


More information about the ffmpeg-devel mailing list