On Thu, Apr 03, 2008 at 11:12:23PM -0700, Junio C Hamano wrote: > >> format-patch already defaults to "--stat --summary -p" if you give it > >> no format options. The default stat is 80, I believe; for format-patch > >> it should probably be bumped to 72 anyway. > [...] > But it loses information, and I am not sure if it is worth it. Why do you > even worry about "a few layers of quoting"? Isn't it a much worse offense > from responder/commenter's part to quote and NOT trimming diffstat part? I don't think a blanket rule to say "it is wrong for a responder to quote the diffstat part" makes sense. I have several times seen people specifically responding to the diffstat. At any rate, I don't think the current setting is a big problem; my thought was that we are generating an email, and that certain line length conventions tend to apply there (though it could just as easily be 76, or 78, or whatever convention). Losing a few characters off the end of the diffstat doesn't really seem like information loss to me (the +++-- part is gets chomped, and it is already being chomped for the 80-character limit; we just normalize it to a slightly smaller number of characters). But I don't really care that much, so please drop the patch if you are concerned about that. -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
