On Sun, Aug 10, 2014 at 06:13:27PM +0300, Alexander Shopov wrote:
> Signed-off-by: Alexander Shopov <ash@xxxxxxxxxxxxxx>
It would probably make sense to put the discussion from your cover
letter into the commit message.
> diff --git a/builtin/index-pack.c b/builtin/index-pack.c
> index 5568a5b..d9c5911 100644
> --- a/builtin/index-pack.c
> +++ b/builtin/index-pack.c
> @@ -1064,7 +1064,7 @@ static void parse_pack_objects(unsigned char *sha1)
> nr_delays--;
> }
> if (nr_delays)
> - die(_("confusion beyond insanity in parse_pack_objects()"));
> + die(_("fatal error in function \"parse_pack_objects\". This is a bug in Git. Please report it to the developers with an e-mail to git@xxxxxxxxxxxxxxx."));
> }
We usually just say:
die("BUG: ...");
here (and hopefully the "..." actually describes the situation a bit
better). I have wondered if we should actually introduce a
BUG("...");
function. Then it would make it simple to be more verbose (e.g.,
pointing the user to the mailing list as you do here) without having to
repeat the text in each place.
-Peff
--
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