From: Jeff King <peff@peff.net>
Date: Fri, 24 Apr 2009 21:28:56 GMT
Subject: Re: [PATCH 1/2] Documentation: replace sha1 by SHA-1
Message-ID: <20090424212856.GB14435@coredump.intra.peff.net>
In-Reply-To: <1240584575-5702-2-git-send-email-git@drmicha.warpmail.net>

On Fri, Apr 24, 2009 at 04:49:34PM +0200, Michael J Gruber wrote:

> --- a/Documentation/git-cat-file.txt
> +++ b/Documentation/git-cat-file.txt
> @@ -76,7 +76,7 @@ If '--batch' is specified, output of the following form is printed for each
>  object specified on stdin:
>  
>  ------------
> -<sha1> SP <type> SP <size> LF
> +<SHA-1> SP <type> SP <size> LF
>  <contents> LF
>  ------------

Maybe it is just me, but I find the original for this one easier to
read. Perhaps because <sha1> is really a variable name here (but for a
human reader to interpret instead of a compiler), so I find the
punctuation and capitalization distracting.

I wonder if all <sha1> should simply be left as-is.

-Peff