From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1753005AbaCGVXp (ORCPT ); Fri, 7 Mar 2014 16:23:45 -0500 Received: from shards.monkeyblade.net ([149.20.54.216]:43550 "EHLO shards.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751792AbaCGVXo (ORCPT ); Fri, 7 Mar 2014 16:23:44 -0500 Date: Fri, 07 Mar 2014 16:23:41 -0500 (EST) Message-Id: <20140307.162341.1600860024275495539.davem@davemloft.net> To: joe@perches.com Cc: alistair@popple.id.au, netdev@vger.kernel.org, linuxppc-dev@lists.ozlabs.org, linux-kernel@vger.kernel.org, devicetree@vger.kernel.org, akpm@linux-foundation.org, error27@gmail.com, josh@joshtriplett.org, randy.dunlap@oracle.com Subject: Re: rfc: checkpatch logical line continuations From: David Miller In-Reply-To: <1394226164.16156.96.camel@joe-AO722> References: <1394077948-8395-3-git-send-email-alistair@popple.id.au> <20140307.154142.488351276799532264.davem@davemloft.net> <1394226164.16156.96.camel@joe-AO722> X-Mailer: Mew version 6.5 on Emacs 24.1 / Mule 6.0 (HANACHIRUSATO) Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit X-Greylist: Sender succeeded SMTP AUTH, not delayed by milter-greylist-4.5.7 (shards.monkeyblade.net [149.20.54.216]); Fri, 07 Mar 2014 13:23:44 -0800 (PST) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Joe Perches Date: Fri, 07 Mar 2014 13:02:44 -0800 > Right now, checkpatch emits a --strict only warning on "&&" or "||" > at the beginning of line but that could be changed to any "$Operators" > > our $Arithmetic = qr{\+|-|\*|\/|%}; > our $Operators = qr{ > <=|>=|==|!=| > =>|->|<<|>>|<|>|!|~| > &&|\|\||,|\^|\+\+|--|&|\||$Arithmetic > }x; > > The ones that likely have a too high false positive rates > are the negation "!" and bitwise "~". Unary operators at the beginning of a line are perfectly fine, it's the other ones that are the problem. From mboxrd@z Thu Jan 1 00:00:00 1970 From: David Miller Subject: Re: rfc: checkpatch logical line continuations Date: Fri, 07 Mar 2014 16:23:41 -0500 (EST) Message-ID: <20140307.162341.1600860024275495539.davem@davemloft.net> References: <1394077948-8395-3-git-send-email-alistair@popple.id.au> <20140307.154142.488351276799532264.davem@davemloft.net> <1394226164.16156.96.camel@joe-AO722> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Content-Transfer-Encoding: 7bit Cc: alistair-Y4h6yKqj69EXC2x5gXVKYQ@public.gmane.org, netdev-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, linuxppc-dev-uLR06cmDAlY/bJ5BZ2RsiQ@public.gmane.org, linux-kernel-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, devicetree-u79uwXL29TY76Z2rM5mHXA@public.gmane.org, akpm-de/tnXTf+JLsfHDXvbKv3WD2FQJk+8+b@public.gmane.org, error27-Re5JQEeQqe8AvxtiuMwx3w@public.gmane.org, josh-iaAMLnmF4UmaiuxdJuQwMA@public.gmane.org, randy.dunlap-QHcLZuEGTsvQT0dZR+AlfA@public.gmane.org To: joe-6d6DIl74uiNBDgjK7y7TUQ@public.gmane.org Return-path: In-Reply-To: <1394226164.16156.96.camel@joe-AO722> Sender: devicetree-owner-u79uwXL29TY76Z2rM5mHXA@public.gmane.org List-Id: netdev.vger.kernel.org From: Joe Perches Date: Fri, 07 Mar 2014 13:02:44 -0800 > Right now, checkpatch emits a --strict only warning on "&&" or "||" > at the beginning of line but that could be changed to any "$Operators" > > our $Arithmetic = qr{\+|-|\*|\/|%}; > our $Operators = qr{ > <=|>=|==|!=| > =>|->|<<|>>|<|>|!|~| > &&|\|\||,|\^|\+\+|--|&|\||$Arithmetic > }x; > > The ones that likely have a too high false positive rates > are the negation "!" and bitwise "~". Unary operators at the beginning of a line are perfectly fine, it's the other ones that are the problem. -- To unsubscribe from this list: send the line "unsubscribe devicetree" in the body of a message to majordomo-u79uwXL29TY76Z2rM5mHXA@public.gmane.org More majordomo info at http://vger.kernel.org/majordomo-info.html From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from shards.monkeyblade.net (shards.monkeyblade.net [IPv6:2001:4f8:3:36:211:85ff:fe63:a549]) by ozlabs.org (Postfix) with ESMTP id 95E772C00B9 for ; Sat, 8 Mar 2014 08:23:45 +1100 (EST) Date: Fri, 07 Mar 2014 16:23:41 -0500 (EST) Message-Id: <20140307.162341.1600860024275495539.davem@davemloft.net> To: joe@perches.com Subject: Re: rfc: checkpatch logical line continuations From: David Miller In-Reply-To: <1394226164.16156.96.camel@joe-AO722> References: <1394077948-8395-3-git-send-email-alistair@popple.id.au> <20140307.154142.488351276799532264.davem@davemloft.net> <1394226164.16156.96.camel@joe-AO722> Mime-Version: 1.0 Content-Type: Text/Plain; charset=us-ascii Cc: randy.dunlap@oracle.com, devicetree@vger.kernel.org, error27@gmail.com, alistair@popple.id.au, linux-kernel@vger.kernel.org, josh@joshtriplett.org, netdev@vger.kernel.org, akpm@linux-foundation.org, linuxppc-dev@lists.ozlabs.org List-Id: Linux on PowerPC Developers Mail List List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , From: Joe Perches Date: Fri, 07 Mar 2014 13:02:44 -0800 > Right now, checkpatch emits a --strict only warning on "&&" or "||" > at the beginning of line but that could be changed to any "$Operators" > > our $Arithmetic = qr{\+|-|\*|\/|%}; > our $Operators = qr{ > <=|>=|==|!=| > =>|->|<<|>>|<|>|!|~| > &&|\|\||,|\^|\+\+|--|&|\||$Arithmetic > }x; > > The ones that likely have a too high false positive rates > are the negation "!" and bitwise "~". Unary operators at the beginning of a line are perfectly fine, it's the other ones that are the problem.