linux-kernel - Re: [PATCH] fixed coding style issues

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] [thread-next>] [day] [month] [year] [list]

Date:	Wed, 25 Jul 2012 16:25:47 +0800
From:	Baodong Chen <chenbdchenbd@...il.com>
To:	Joe Perches <joe@...ches.com>
Cc:	dmitry.torokhov@...il.com, linux-input@...r.kernel.org,
	linux-kernel@...r.kernel.org
Subject: Re: [PATCH] fixed coding style issues

On Wed, Jul 25, 2012 at 4:01 PM, Joe Perches <joe@...ches.com> wrote:
> On Wed, 2012-07-25 at 15:44 +0800, Baodong Chen wrote:
>> i want to participate in and to start from
>> fixing simple issues!
>
> That's fine Baodong.  Welcome.
>
>> how could i choose issues form checkpatch's output to find which
>> should be fixed which are
>> not?
>
> checkpatch output is merely a guide for submission of patches
> to conform to some generic "kernel style".  I think there are
> times when really ugly code should have a pass to make it more
> "kernel style" like.  Most of those cleanup type changes should
> be done only to code in the "drivers/staging" directory.
>
> I think most of the time, unless you are also fixing some other
> underlying code defect or shortcoming, checkpatch only cleanups
> to existing code are best avoided.
>
> I suggest for now, until you are really comfortable with the
> preferred style, you confine yourself to drivers/staging.
>
hi,Joe;
thank for your explaining!
ok, i will start from drivers/staging

>> or checkpatch.pl should be updated??
>
> You could work on that too.
>
> One possibility is to write a utility to take checkpatch output
> and generate patches automatically, or perhaps extend checkpatch
> itself to do that.  (ie: add a --fix option)
>
> cheers, Joe
>
--
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/