On 03/10/2014 08:46 AM, Michael Haggerty wrote:
> Instead of, for example,
>
> fatal: update refs/heads/master missing [<oldvalue>] NUL
>
> emit
>
> fatal: update refs/heads/master missing <oldvalue>
[snip]
> - die("update %s missing [<oldvalue>] NUL", update->ref_name);
> + die("update %s missing <oldvalue>", update->ref_name);
The reason for the original wording is that the <oldvalue> is indeed
optional. This can only occur at end-of-input, and it is actually the
*NUL* that is missing because an empty old value can be specified to
mean that it it intentionally missing.
-Brad
--
To unsubscribe from this list: send the line "unsubscribe git" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html