From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1759393AbdEYOB3 (ORCPT ); Thu, 25 May 2017 10:01:29 -0400 Received: from mail-wm0-f41.google.com ([74.125.82.41]:35518 "EHLO mail-wm0-f41.google.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1751600AbdEYOB0 (ORCPT ); Thu, 25 May 2017 10:01:26 -0400 From: Michal Nazarewicz To: Ian Abbott , linux-kernel@vger.kernel.org, linux-arch@vger.kernel.org Cc: Alexander Potapenko , Andrew Morton , Arnd Bergmann , Borislav Petkov , Hidehiro Kawai , Ian Abbott , Jakub Kicinski , Johannes Berg , Kees Cook , Peter Zijlstra , Rasmus Villemoes , Steven Rostedt Subject: Re: [PATCH v5 4/6] linux/bug.h: correct "space required before that '-'" In-Reply-To: <20170525120316.24473-5-abbotti@mev.co.uk> Organization: http://mina86.com/ References: <20170525120316.24473-1-abbotti@mev.co.uk> <20170525120316.24473-5-abbotti@mev.co.uk> Face: iVBORw0KGgoAAAANSUhEUgAAADAAAAAwBAMAAAClLOS0AAAAJFBMVEWbfGlUPDDHgE57V0jUupKjgIObY0PLrom9mH4dFRK4gmjPs41MxjOgAAACP0lEQVQ4T23Sv2vbQBQHcBk1xE6WyALX107VUEgmn6+ouUwpEQQ6uRjttkWP4CkBg2M0BQLBdPFZYPsyFYo7qEtKDQ7on+t7+nF2Ux8ahD587717OmNYrOvycHsZ+o2r051wHTHysAvGb8ygvgu4QWT0sCmkgZCIEnlV2X8BtyraazFGDuxhmKSQJMlwHQ7v5MHSNxmz78rfElwAa3ieVD9e+hBhjaPDDG6NgFo2f4wBMNIo5YmRtF0RyDgFjJjlMIWbnuM4x9MMfABGTlN4qgIQB4A1DEyA1BHWtfeWNUMwiVJKoqh97KrkOO+qzgluVYLvFCUKAX73nONeBr7BGMdM6Sg0kuep03VywLaIzRiVr+GAzKlpQIsAFnWAG2e6DT5WmWDiudZMIc6hYrMOmeMQK9WX0B+/RfjzL9DI7Y9/Iayn29Ci0r2i4f9gMimMSZLCDMalgQGU5hnUtqAN0OGvEmO1Wnl0C0wWSCEHnuHBqmygxdxA8oWXwbipoc1EoNR9DqOpBpOJrnr0criQab9ZT4LL+wI+K7GBQH30CrhUruilgP9DRTrhVWZCiAyILP+wiuLeCKGTD6r/nc8LOJcAwR6IBTUs+7CASw3QFZ0MdA2PI3zNziH4ZKVhXCRMBjeZ1DWMekKwDCASwExy+NQ86TaykaDAFHO4aP48y4fIcDM5yOG8GcTLbOyp8A8azjJI93JFd1EA6yN8sSxMQJWoABqniRZVykYgRXErzrdqExAoUrRb0xfRp8p2A/4XmfilTtkDZ4cAAAAASUVORK5CYII= X-Face: -TR8(rDTHy/(xl?SfWd1|3:TTgDIatE^t'vop%*gVg[kn$t{EpK(P"VQ=~T2#ysNmJKN$"yTRLB4YQs$4{[.]Fc1)*O]3+XO^oXM>Q#b^ix,O)Zbn)q[y06$`e3?C)`CwR9y5riE=fv^X@x$y?D:XO6L&x4f-}}I4=VRNwiA^t1-ZrVK^07.Pi/57c_du'& OpenPGP: id=AC1F5F5CD41888F8CC8458582060401250751FF4; url=http://mina86.com/mina86.pub X-Hashcash: 1:20:170525:rostedt@goodmis.org::tGY9Mz3TSI+ghKcQ:00000000000000000000000000000000000000000003rg X-Hashcash: 1:20:170525:linux-arch@vger.kernel.org::v5WOqpzP1bkBshOj:000000000000000000000000000000000001Q/0 X-Hashcash: 1:20:170525:linux-kernel@vger.kernel.org::Idd8d1lzYZ+/L2oi:0000000000000000000000000000000001yfN X-Hashcash: 1:20:170525:abbotti@mev.co.uk::i9x7/2CdlhuTVHMd:000000000000000000000000000000000000000000002Rrw X-Hashcash: 1:20:170525:jakub.kicinski@netronome.com::L3+LnX6CbGYkJOy5:0000000000000000000000000000000002Vy/ X-Hashcash: 1:20:170525:abbotti@mev.co.uk::2viuQP5uR0nZU9mY:000000000000000000000000000000000000000000001q16 X-Hashcash: 1:20:170525:peterz@infradead.org::Vm1HStbP5HXYYmHq:000000000000000000000000000000000000000003Ka8 X-Hashcash: 1:20:170525:akpm@linux-foundation.org::W7zE1dVPQjIliz/5:0000000000000000000000000000000000005PJi X-Hashcash: 1:20:170525:linux@rasmusvillemoes.dk::EZwVqjGAz87nAHVq:00000000000000000000000000000000000004lbI X-Hashcash: 1:20:170525:johannes.berg@intel.com::ndc9OiWfj8o5vHDa:000000000000000000000000000000000000005GHW X-Hashcash: 1:20:170525:keescook@chromium.org::BdE/Rmnj0XPOAFao:000000000000000000000000000000000000000069Et X-Hashcash: 1:20:170525:hidehiro.kawai.ez@hitachi.com::08eAhnb8cJkrkp95:0000000000000000000000000000000071I5 X-Hashcash: 1:20:170525:arnd@arndb.de::w/lUSm++Xb4oUgXs:00007ksa X-Hashcash: 1:20:170525:bp@suse.de::HnnCsBc1vsROZd1J:00000007vdB X-Hashcash: 1:20:170525:glider@google.com::i+5V7SBul22KhdDm:000000000000000000000000000000000000000000008Fxp Date: Thu, 25 May 2017 16:01:21 +0200 Message-ID: MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Content-Transfer-Encoding: 8bit X-MIME-Autoconverted: from quoted-printable to 8bit by mail.home.local id v4PE1iYI013513 On Thu, May 25 2017, Ian Abbott wrote: > Correct these checkpatch.pl errors: > > |ERROR: space required before that '-' (ctx:OxO) > |#37: FILE: include/linux/bug.h:37: > |+#define BUILD_BUG_ON_ZERO(e) (sizeof(struct { int:-!!(e); })) > > |ERROR: space required before that '-' (ctx:OxO) > |#38: FILE: include/linux/bug.h:38: > |+#define BUILD_BUG_ON_NULL(e) ((void *)sizeof(struct { int:-!!(e); })) > > I decided to wrap the bitfield expressions that begin with minus signs > in parentheses rather than insert spaces before the minus signs. > > Signed-off-by: Ian Abbott > Cc: Andrew Morton > Cc: Kees Cook > Cc: Steven Rostedt > Cc: Peter Zijlstra > Cc: Jakub Kicinski > Cc: Rasmus Villemoes Acked-by: Michal Nazarewicz > --- > v5: Actually, there was no v1 thru v4. I called this v5 to match the > series. > --- > include/linux/bug.h | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/include/linux/bug.h b/include/linux/bug.h > index 216a1b79653d..483207cb99fb 100644 > --- a/include/linux/bug.h > +++ b/include/linux/bug.h > @@ -36,8 +36,8 @@ struct pt_regs; > * e.g. in a structure initializer (or where-ever else comma expressions > * aren't permitted). > */ > -#define BUILD_BUG_ON_ZERO(e) (sizeof(struct { int:-!!(e); })) > -#define BUILD_BUG_ON_NULL(e) ((void *)sizeof(struct { int:-!!(e); })) > +#define BUILD_BUG_ON_ZERO(e) (sizeof(struct { int:(-!!(e)); })) > +#define BUILD_BUG_ON_NULL(e) ((void *)sizeof(struct { int:(-!!(e)); })) > > /* > * BUILD_BUG_ON_INVALID() permits the compiler to check the validity of the -- Best regards ミハウ “𝓶𝓲𝓷𝓪86” ナザレヴイツ «If at first you don’t succeed, give up skydiving»