Message-ID: <1380138515.17366.22.camel@joe-AO722>
Subject: Re: [PATCH] checkpatch: Find CamelCase definitions of
 struct/union/enum
From: Joe Perches <joe@perches.com>
To: "Theodore Ts'o" <tytso@mit.edu>
Cc: Al Viro <viro@ZenIV.linux.org.uk>,
        Andrew Morton <akpm@linux-foundation.org>,
        Andy Whitcroft <apw@shadowen.org>, LKML <linux-kernel@vger.kernel.org>
Date: Wed, 25 Sep 2013 12:48:35 -0700
In-Reply-To: <20130925193035.GA11905@thunk.org>
References: <1380047934.3575.100.camel@joe-AO722>
	 <20130925152432.GY13318@ZenIV.linux.org.uk>
	 <1380123344.17366.5.camel@joe-AO722>
	 <20130925161949.GZ13318@ZenIV.linux.org.uk>
	 <1380136920.17366.15.camel@joe-AO722> <20130925193035.GA11905@thunk.org>
Content-Type: text/plain; charset="ISO-8859-1"
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Sender: linux-kernel-owner@vger.kernel.org
Content-Length: 1662
Lines: 53

On Wed, 2013-09-25 at 15:30 -0400, Theodore Ts'o wrote:
> On Wed, Sep 25, 2013 at 12:22:00PM -0700, Joe Perches wrote:
> > 
> > > 	* labels a-la Enomem, etc.  I've been using those and I will
> > > keep doing so, checkpatch.pl and its users be damned.
> > 
> > Good on you.
> 
> I'm with Al.  This is just silly to add to checkpatch.
> 
> > Fine by me.  Don't use checkpatch.  No stress from me.
> 
> I still get some of your minions sending patches asking to "fix" ext4
> from time to time.  It wastes everyone's time.

ext4 still has several code style nits that could be
improved.  Up to you to do anything about them though.

no space after comma, pointer spaces, and split strings
are nits.

> BTW, checkpatch barfs on pretty much everything in
> linux/trace/events/*.h

include/trace/events/*.h

As I've said multiple times, checkpatch is a stupid
little tool that can be ignored by anyone with
different tastes.

And, shrug.  I think straining for alignment like below
doesn't really add much.

$ ./scripts/checkpatch.pl -f include/trace/events/jbd.h
ERROR: space prohibited after that open parenthesis '('
#17: FILE: trace/events/jbd.h:17:
+		__field(	dev_t,	dev			)

ERROR: space prohibited before that close parenthesis ')'
#17: FILE: trace/events/jbd.h:17:
+		__field(	dev_t,	dev			)

ERROR: space prohibited after that open parenthesis '('
#18: FILE: trace/events/jbd.h:18:
+		__field(	int,	result			)


--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/