On Thu, Feb 19, 2009 at 01:53:10PM +0100, Miklos Vajna wrote: > @@ -2932,8 +2932,7 @@ static int write_out_one_reject(struct patch *patch) > cnt = strlen(patch->new_name); > if (ARRAY_SIZE(namebuf) <= cnt + 5) { > cnt = ARRAY_SIZE(namebuf) - 5; > - fprintf(stderr, > - "warning: truncating .rej filename to %.*s.rej", > + warning("truncating .rej filename to %.*s.rej", > cnt - 1, patch->new_name); > } Hmm. This hunk is actually changing behavior, as the original didn't have a newline. It looks like a bug in the original, though. Other than that, these all look pretty straightforward. Probably the shell scripts should be switched to match, too. But it would be nice to hear from Junio first that this cleanup is even desired (so you don't waste time). -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