Re: [PATCH] kernel: acct.c: spaces required around that '<'

From: Al Viro
Date: Thu Jul 19 2012 - 18:55:48 EST


On Thu, Jul 19, 2012 at 10:10:27PM +0100, jan.bannister@xxxxxxxxx wrote:
> From: Jan Bannister <jan.bannister@xxxxxxxxx>
>
> Fixed a coding style issue

NAK, because I'm sick and tired of the pompous commit messages. I'm fine
with adding those spaces. However, I _do_ have a lot against the commit
messages like that. "Required"? By whom? Holy checkpatch.pl? "Coding
style issue" - completely uninformative; there all kinds of those.

FWIW, my prefernce would be something along the lines of "whitespace
tidy-up in <function or file name>". At least that says what happens
in commit and where is it happening...
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/