Re: [PATCH 1/3] revision: clarify a 'return NULL' in get_reference()

[Eric Sunshine <sunshine@xxxxxxxxxxxxxx>]

On Thu, Feb 1, 2024 at 6:58 AM Christian Couder
<christian.couder@xxxxxxxxx> wrote:
> In general when we know a pointer variable is NULL, it's clearer to
> explicitely return NULL than to return that variable.

s/explicitely/explicitly/

> 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 explicitely
> returning NULL when 'revs->ignore_missing' is true.

s/explicitely/explicitly/

> Signed-off-by: Christian Couder <chriscool@xxxxxxxxxxxxx>