linux-kernel - Re: [PATCH] scripts/cvt_kernel_style.pl: kernel style source code reformatter

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 for Android: free password hash cracker in your pocket

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

Message-Id: <201003242332.00074.elendil@planet.nl>
Date:	Wed, 24 Mar 2010 23:31:58 +0100
From:	Frans Pop <elendil@...net.nl>
To:	Joe Perches <joe@...ches.com>
Cc:	linux-kernel@...r.kernel.org, Greg KH <gregkh@...e.de>,
	devel@...verdev.osuosl.org, apw@...dowen.org,
	Andrew Morton <akpm@...ux-foundation.org>
Subject: Re: [PATCH] scripts/cvt_kernel_style.pl: kernel style source code reformatter

On Wednesday 24 March 2010, Joe Perches wrote:
> A script to convert kernel source files to a more conformant style.
> A supplement to or replacement of Lindent.
> A wretched little perl script using regexes.

I've found the script useful, but it does require some experience and 
judgement to use correctly.

For example, with my 'trailing spaces in messages' patches I've found it
is essential to review all changes to avoid e.g. changing output to files 
in /proc. Doing that can be valid, but it probably should not be included 
in the same patch as removing trailing space from kernel messages.

Given that experience and judgement is needed to use many tools correctly,

Supported-by: Frans Pop <elendil@...net.nl>

Cheers,
FJP
--
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/