Re: [PATCH] Removed redundant static functions such as update_tracking_ref() and verify_remote_names() from builtin-send-pack.c, and made the ones in transport.c not be static so they can be used instead.
- From
Jeff King <peff@peff.net>
- Date
- Apr 24, 2009, 21:23 UTC
- Message-ID
- <20090424212313.GA14435@coredump.intra.peff.net>
- In-Reply-To
- <99B4BF12-01B9-4A68-B2E0-EF5DF2595FF0@petdance.com>
On Fri, Apr 24, 2009 at 04:13:14PM -0500, Andy Lester wrote:
> This was what I was looking for. I think what I'll do is fold your > message into Documentation/SubmittingPatches and submit that as a patch > first.
That probably makes sense.
I keep thinking about writing a separate "how to write a good commit message" document that would be more universal than just "here's how you submit a patch to git". And some of what I wrote to you could probably go in such a document. But I don't know if it makes sense to start a new document just with what I said there; it might be a bit sparse (OTOH, maybe people would then be encouraged to add their tips to it).
-Peff