[FFmpeg-devel] [PATCH 4/7] avcodec/h261dec: Remove pointless goto

Andreas Rheinhardt andreas.rheinhardt at outlook.com
Sat Oct 7 03:40:28 EEST 2023


There is no need to parse the header twice; doing so does nothing.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at outlook.com>
---
 libavcodec/h261dec.c | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/libavcodec/h261dec.c b/libavcodec/h261dec.c
index 91d555a70e..272bbbea34 100644
--- a/libavcodec/h261dec.c
+++ b/libavcodec/h261dec.c
@@ -605,7 +605,6 @@ static int h261_decode_frame(AVCodecContext *avctx, AVFrame *pict,
 
     h->gob_start_code_skipped = 0;
 
-retry:
     init_get_bits(&s->gb, buf, buf_size * 8);
 
     ret = h261_decode_picture_header(h);
@@ -627,8 +626,6 @@ retry:
         ret = ff_set_dimensions(avctx, s->width, s->height);
         if (ret < 0)
             return ret;
-
-        goto retry;
     }
 
     if ((avctx->skip_frame >= AVDISCARD_NONREF && s->pict_type == AV_PICTURE_TYPE_B) ||
-- 
2.34.1



More information about the ffmpeg-devel mailing list