| 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
| ||
|
Message-ID: <alpine.LNX.2.00.0907211139400.2147@iabervon.org> Date: Tue, 21 Jul 2009 11:48:37 -0400 (EDT) From: Daniel Barkalow <barkalow@...ervon.org> To: Pavel Machek <pavel@....cz> cc: apw@...onical.com, kernel list <linux-kernel@...r.kernel.org> Subject: Re: checkpatch whitespace On Tue, 21 Jul 2009, Pavel Machek wrote: > I don't think that a bit of trailing whitespace hurts that much (it is > very easy to create when fixing overly long lines, another checkpatch > complain), and at least akpm autofixes the whitespace, anyway, but... The problem with trailing whitespace in files is that future developers have to keep it the same while making changes in the area, or it'll add noise to their commits and patches. Fixing up a patch to stop removing trailing whitespace is even more of a pain than fixing a patch to stop adding it. > Would it be possible to add autofixing to checkpatch? Fixing > whitespace issues by hand is useless & boring task... Until checkpatch does it, "git apply" can be configured to auto-correct. -Daniel *This .sig left intentionally blank* -- To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists