http-push: avoid a needless goto
authorJohannes Schindelin <Johannes.Schindelin@gmx.de>
Thu, 14 Feb 2008 23:26:12 +0000 (23:26 +0000)
committerJunio C Hamano <gitster@pobox.com>
Sat, 16 Feb 2008 07:21:57 +0000 (23:21 -0800)
There was a goto, and while it is not half as harmful as some people
believe, it was unnecessary here.  So remove it for readability.

Signed-off-by: Johannes Schindelin <johannes.schindelin@gmx.de>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
http-push.c

index 14ef652ca76a6abab44e5f5e5b3265c3cdb34506..f9b77d6021cf8df4b4d7a7a1efea3e63b5dcb477 100644 (file)
@@ -2413,12 +2413,9 @@ int main(int argc, char **argv)
                } while (request_queue_head && !aborted);
 
                /* Update the remote branch if all went well */
-               if (aborted || !update_remote(ref->new_sha1, ref_lock)) {
+               if (aborted || !update_remote(ref->new_sha1, ref_lock))
                        rc = 1;
-                       goto unlock;
-               }
 
-       unlock:
                if (!rc)
                        fprintf(stderr, "    done\n");
                unlock_remote(ref_lock);