linux-kernel - Re: [PATCH] checkpatch.pl: Add warning for new _

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]

Message-ID: <1393452256.24588.5.camel@joe-AO722>
Date:	Wed, 26 Feb 2014 14:04:16 -0800
From:	Joe Perches <joe@...ches.com>
To:	Tom Rini <trini@...com>
Cc:	Josh Triplett <josh@...htriplett.org>,
	linux-kernel@...r.kernel.org,
	Andrew Morton <akpm@...ux-foundation.org>
Subject: Re: [PATCH] checkpatch.pl: Add warning for new __packed additions

On Tue, 2014-02-25 at 07:30 -0500, Tom Rini wrote:
> The only intentional differences we have today are adding debug/printf
> to the list of log functions

That seems fine and trivial to keep current.

> and pointing people at boards.cfg not
> CHECKPATCH in MAINTAINERS,

That seems wrong.

The idea of that message is to fix defects in checkpatch,
not any other code.

Maybe that could be reworded to
"See the CHECKPATCH entry in the MAINTAINERS file of linux git"
or some such.

>  so I can re-sync us no problem as, yup, we're
> about a year out of date there.  Oh and I see now someone snuck in a
> check for whitespace before semi-colon to catch 'return 0 ;' which I'll
> check real quick isn't something already caught and if not post that as
> well.

Does anyone know of other projects that use checkpatch?


--
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/