On 12/11/24 5:32 PM, Marth64 wrote:
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?
They could be. There use to be a free statement there that was moved,
and I didn't update the gotos. Good catch!
- Leo Izen (Traneptora)
_______________________________________________
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".
