linux-kernel - Re: [PATCH v4] Add .editorconfig file for basic formatting

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Date: Tue, 11 Jun 2024 08:44:40 +0200
From: Greg KH <gregkh@...uxfoundation.org>
To: Jason Gunthorpe <jgg@...dia.com>
Cc: Mateusz Guzik <mjguzik@...il.com>,
	Masahiro Yamada <masahiroy@...nel.org>,
	Íñigo Huguet <ihuguet@...hat.com>,
	Vincent MAILHOL <mailhol.vincent@...adoo.fr>,
	Jonathan Corbet <corbet@....net>, ojeda@...nel.org,
	danny@...ag0n.dev, mic@...ikod.net, linux-kernel@...r.kernel.org,
	joe@...ches.com, linux@...musvillemoes.dk, willy@...radead.org
Subject: Re: [PATCH v4] Add .editorconfig file for basic formatting

On Mon, Jun 03, 2024 at 09:18:53AM -0300, Jason Gunthorpe wrote:
> On Sun, Jun 02, 2024 at 05:30:51PM +0200, Mateusz Guzik wrote:
> 
> > As is I think the config mostly gets in the way and most people have a
> > setup which does not use it (for example my vim does not). Alternatively
> > maye it is neovim which is overzalous here and other editors don't treat
> > the entire file. If there is a way to convince the thing to only tend to
> > whitespace issues in edited lines that would be great.
> 
> Yes, I think that is quite overzealous, I'd view it as a vim issue.

What?  No, vim is doing exactly what it is asked for here, it "trims all
whitespace" when saving the file.

I too just ran into this today, and it's a pain.  This needs to be
fixed, sorry.

greg k-h