Re: [PATCH] diffcore-pickaxe: simplify has_changes and contains
From: Junio C Hamano <hidden> Date: 2016-06-15 22:58:01
Jeff King [off-list ref] writes:
Before, if (!one && !two) we would call contains(two, ...), and now we
will simply assume it is zero. Which I think is an improvement, as we
would have segfaulted before. I don't think it is a bug in the current
code (we would not ever feed the function two NULLs), but it is nice to
be more defensive.
Acked-by: Jeff King <redacted>