9 Apr
2012
9 Apr
'12
10:17 p.m.
On Mon, 9 Apr 2012, Jiri Kosina wrote:
On Mon, 9 Apr 2012, Jesper Juhl wrote:
While reading through sound/soc/codecs/wm8994.c I noticed a fair amount of trailing whitespace. This patch gets rid of it.
Signed-off-by: Jesper Juhl jj@chaosbits.net
Jesper,
is it really worth it?
Maybe not. I just noticed it while I was reading through the file for other purposes and it annoyed me.
The code should have been fixed before merging, yes, but once it's there, such fixups tend to only make people angry (because they have to do unnecessary extra steps during git blame).
I think it's nice to get little things like this cleaned up, but at the same time I don't want to annoy other people. I won't yell if you decide to just drop this patch.
--
Jesper Juhl jj@chaosbits.net http://www.chaosbits.net/
Don't top-post http://www.catb.org/jargon/html/T/top-post.html
Plain text mails only, please.