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>