lkml.org 
[lkml]   [2021]   [Mar]   [18]   [last100]   RSS Feed
Views: [wrap][no wrap]   [headers]  [forward] 
 
Messages in this thread
/
From
Date
SubjectRe: [PATCH master] module: remove never implemented MODULE_SUPPORTED_DEVICE
On Thu, Mar 18, 2021 at 10:49 AM Leon Romanovsky <leon@kernel.org> wrote:
>
> No, I opened patch and added the note manually, so it is definitely my VIM.
> Most likely this part of my .vimrc caused it.

Ok, that would do it.

Yeah, whitespace is easy to "fix" at patch application time, but it
really is meaningful and you never should change whitespace for
patches.

Maybe you can limit your rules to just particular file types
(although, honestly, I think it's bad for headers and C files too when
it then causes entirely irrelevant and independent changes - you only
want your own _new_ edits to be whitespace-clean, not fix other random
issues).

Better yet, maybe not "fix whitespace" at all, but have some code
coloring logic that just points out bad whitespace? I use "git diff"
myself, with colorization being default for tty operations:

[color]
ui=auto

so that then "git diff" will show you your (new) evil whitespace
errors when you review your changes, but won't complain about existing
whitespace issues..

Linus

\
 
 \ /
  Last update: 2021-03-18 18:57    [W:0.049 / U:0.744 seconds]
©2003-2020 Jasper Spaans|hosted at Digital Ocean and TransIP|Read the blog|Advertise on this site