diff mbox series

[FFmpeg-devel,7/8] avcodec/smacker: Use same variable for return values and errors

Message ID 20200731112241.8948-7-andreas.rheinhardt@gmail.com
State Accepted
Commit e028e8aa39a430568b486751cf82b3a26e24e7af
Headers show
Series [FFmpeg-devel,1/8] avcodec/smacker: Remove write-only and unused variables | expand

Checks

Context Check Description
andriy/default pending
andriy/make success Make finished
andriy/make_fate success Make fate finished

Commit Message

Andreas Rheinhardt July 31, 2020, 11:22 a.m. UTC 
smacker_decode_header_tree() uses different variables for return values
(res) and for errors (err) leading to code like
res = foo(bar);
if (res < 0) {
    err = res;
    goto error;
}
Given that no positive return value is ever used at all one can simplify
the above by removing the intermediate res.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
---
 libavcodec/smacker.c | 21 ++++++++-------------
 1 file changed, 8 insertions(+), 13 deletions(-)

Comments

Paul B Mahol Aug. 19, 2020, 7:21 p.m. UTC | #1 
On 7/31/20, Andreas Rheinhardt <andreas.rheinhardt@gmail.com> wrote:
> smacker_decode_header_tree() uses different variables for return values
> (res) and for errors (err) leading to code like
> res = foo(bar);
> if (res < 0) {
>     err = res;
>     goto error;
> }
> Given that no positive return value is ever used at all one can simplify
> the above by removing the intermediate res.
>

LGTM
diff mbox series

Patch

diff --git a/libavcodec/smacker.c b/libavcodec/smacker.c
index 4b1f0f1b7c..07fa8887d8 100644
--- a/libavcodec/smacker.c
+++ b/libavcodec/smacker.c
@@ -182,13 +182,12 @@  static int smacker_decode_bigtree(GetBitContext *gb, HuffContext *hc,
  */
 static int smacker_decode_header_tree(SmackVContext *smk, GetBitContext *gb, int **recodes, int *last, int size)
 {
-    int res;
     HuffContext huff;
     HuffContext h[2] = { 0 };
     VLC vlc[2] = { { 0 } };
     int escapes[3];
     DBCtx ctx;
-    int err = 0;
+    int err;
 
     if(size >= UINT_MAX>>4){ // (((size + 3) >> 2) + 3) << 2 must not overflow
         av_log(smk->avctx, AV_LOG_ERROR, "size too large\n");
@@ -210,20 +209,17 @@  static int smacker_decode_header_tree(SmackVContext *smk, GetBitContext *gb, int
                    i ? "high" : "low");
             continue;
         }
-        res = smacker_decode_tree(gb, &h[i], 0, 0);
-        if (res < 0) {
-            err = res;
+        err = smacker_decode_tree(gb, &h[i], 0, 0);
+        if (err < 0)
             goto error;
-        }
         skip_bits1(gb);
         if (h[i].current > 1) {
-            res = init_vlc(&vlc[i], SMKTREE_BITS, h[i].length,
+            err = init_vlc(&vlc[i], SMKTREE_BITS, h[i].length,
                            INIT_VLC_DEFAULT_SIZES(h[i].lengths),
                            INIT_VLC_DEFAULT_SIZES(h[i].bits),
                            INIT_VLC_LE);
-            if(res < 0) {
+            if (err < 0) {
                 av_log(smk->avctx, AV_LOG_ERROR, "Cannot build VLC table\n");
-                err = res;
                 goto error;
             }
         }
@@ -253,16 +249,15 @@  static int smacker_decode_header_tree(SmackVContext *smk, GetBitContext *gb, int
     }
     *recodes = huff.values;
 
-    res = smacker_decode_bigtree(gb, &huff, &ctx, 0);
-    if (res < 0) {
-        err = res;
+    err = smacker_decode_bigtree(gb, &huff, &ctx, 0);
+    if (err < 0)
         goto error;
-    }
     skip_bits1(gb);
     if(ctx.last[0] == -1) ctx.last[0] = huff.current++;
     if(ctx.last[1] == -1) ctx.last[1] = huff.current++;
     if(ctx.last[2] == -1) ctx.last[2] = huff.current++;
 
+    err = 0;
 error:
     for (int i = 0; i < 2; i++) {
         if (vlc[i].table)