"Horst H. von Brand" <vonbrand@xxxxxxxxxxxx> wrote: > diff --git a/Documentation/git-receive-pack.txt b/Documentation/git-receive-pack.txt > index 10e8c46..2fafc79 100644 > --- a/Documentation/git-receive-pack.txt > +++ b/Documentation/git-receive-pack.txt > @@ -20,7 +20,7 @@ The UI for the protocol is on the 'git-send-pack' side, and the > program pair is meant to be used to push updates to remote > repository. For pull operations, see 'git-fetch-pack'. > > -The command allows for creation and fast forwarding of sha1 refs > +The command allows for creation and fast forwarding of SHA-1 refs A good cleanup and correction. We've apparently been a little lax. Its a noble and worthy goal to make the correction and I applaud you for doing the hard work. > diff --git a/builtin-apply.c b/builtin-apply.c > index 3fefdac..814f78f 100644 > --- a/builtin-apply.c > +++ b/builtin-apply.c > @@ -1811,7 +1812,8 @@ static int apply_binary(struct buffer_desc *desc, struct patch *patch) > "without full index line", name); > > if (patch->old_name) { > - /* See if the old one matches what the patch > + /* > + * See if the old one matches what the patch > * applies to. > */ > hash_sha1_file(desc->buffer, desc->size, blob_type, sha1); However I cannot help but feel that this hunk is unrelated to the theme of this extremely large patch. I don't know how Junio feels, but this late in the 1.5.0 series I'm a little leary of a 1600+ line patch which is changing so much code, even if its something as trivial as the above hunk. -- Shawn. - 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