diff options
author | Christian Couder <christian.couder@gmail.com> | 2024-02-14 14:25:10 (GMT) |
---|---|---|
committer | Junio C Hamano <gitster@pobox.com> | 2024-02-14 17:38:22 (GMT) |
commit | 3ff56af99b1c9becd9e603b59986359f553e62a8 (patch) | |
tree | 08b10d2d735bfa9b65f48ee2d540b3cec3b172b9 /revision.c | |
parent | 75389e275c288a328272b70e841ada767313a739 (diff) | |
download | git-3ff56af99b1c9becd9e603b59986359f553e62a8.zip git-3ff56af99b1c9becd9e603b59986359f553e62a8.tar.gz git-3ff56af99b1c9becd9e603b59986359f553e62a8.tar.bz2 |
revision: clarify a 'return NULL' in get_reference()
When we know a pointer variable is NULL, it's clearer to
explicitly return NULL than to return that variable.
In get_reference(), when 'object' is NULL, we already return NULL
when 'revs->exclude_promisor_objects && is_promisor_object(oid)' is
true, but we return 'object' when 'revs->ignore_missing' is true.
Let's make the code clearer and more uniform by also explicitly
returning NULL when 'revs->ignore_missing' is true.
Helped-by: Eric Sunshine <sunshine@sunshineco.com>
Signed-off-by: Christian Couder <chriscool@tuxfamily.org>
Signed-off-by: Junio C Hamano <gitster@pobox.com>
Diffstat (limited to 'revision.c')
-rw-r--r-- | revision.c | 2 |
1 files changed, 1 insertions, 1 deletions
@@ -385,7 +385,7 @@ static struct object *get_reference(struct rev_info *revs, const char *name, if (!object) { if (revs->ignore_missing) - return object; + return NULL; if (revs->exclude_promisor_objects && is_promisor_object(oid)) return NULL; die("bad object %s", name); |