summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorJeff King <peff@peff.net>2014-01-08 04:35:00 -0500
committerJunio C Hamano <gitster@pobox.com>2014-01-17 09:34:29 -0800
commit2a9b7a90ed2bd49aaea632cfe5185b878bde9104 (patch)
tree6562092abff9e0fbbaa6058562456b634858c162
parent61418263e8fe9ba86292b91b6952e083d5d77c5c (diff)
downloadgit-2a9b7a90ed2bd49aaea632cfe5185b878bde9104.tar.gz
branch_get: return early on error
Right now we simply check if "ret" is valid before doing further processing. As we add more processing, this will become more and more cumbersome. Instead, let's just check whether "ret" is invalid and return early with the error. Signed-off-by: Jeff King <peff@peff.net> Signed-off-by: Junio C Hamano <gitster@pobox.com>
-rw-r--r--remote.c5
1 files changed, 4 insertions, 1 deletions
diff --git a/remote.c b/remote.c
index 9f1a8aa2c4..e18e2fd38f 100644
--- a/remote.c
+++ b/remote.c
@@ -1512,7 +1512,10 @@ struct branch *branch_get(const char *name)
ret = current_branch;
else
ret = make_branch(name, 0);
- if (ret && ret->remote_name) {
+ if (!ret)
+ return NULL;
+
+ if (ret->remote_name) {
ret->remote = remote_get(ret->remote_name);
if (ret->merge_nr) {
int i;