[FFmpeg-cvslog] avcodec/decode: Return early upon failure

Andreas Rheinhardt git at videolan.org
Fri Mar 5 15:33:49 EET 2021


ffmpeg | branch: master | Andreas Rheinhardt <andreas.rheinhardt at gmail.com> | Thu Mar  4 09:32:21 2021 +0100| [684e501cb9f1fb001228770d4e892f647de15202] | committer: Andreas Rheinhardt

avcodec/decode: Return early upon failure

Allows to remove one level of indentation.

Reviewed-by: James Almer <jamrial at gmail.com>
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=684e501cb9f1fb001228770d4e892f647de15202
---

 libavcodec/decode.c | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

diff --git a/libavcodec/decode.c b/libavcodec/decode.c
index 5e5d0c7066..db6ee9cb04 100644
--- a/libavcodec/decode.c
+++ b/libavcodec/decode.c
@@ -1043,9 +1043,9 @@ int avcodec_decode_subtitle2(AVCodecContext *avctx, AVSubtitle *sub,
         AVPacket pkt_recoded = *avpkt;
 
         ret = recode_subtitle(avctx, &pkt_recoded, avpkt);
-        if (ret < 0) {
-            *got_sub_ptr = 0;
-        } else {
+        if (ret < 0)
+            return ret;
+
              ret = extract_packet_props(avctx->internal, &pkt_recoded);
              if (ret < 0)
                 return ret;
@@ -1099,7 +1099,6 @@ int avcodec_decode_subtitle2(AVCodecContext *avctx, AVSubtitle *sub,
 
                 av_packet_unref(&pkt_recoded);
             }
-        }
 
         if (*got_sub_ptr)
             avctx->frame_number++;



More information about the ffmpeg-cvslog mailing list