diff options
author | Jeff King <peff@peff.net> | 2014-01-08 04:35:00 -0500 |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2014-01-17 09:34:29 -0800 |
commit | 2a9b7a90ed2bd49aaea632cfe5185b878bde9104 (patch) | |
tree | 6562092abff9e0fbbaa6058562456b634858c162 | |
parent | 61418263e8fe9ba86292b91b6952e083d5d77c5c (diff) | |
download | git-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.c | 5 |
1 files changed, 4 insertions, 1 deletions
@@ -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; |