From mboxrd@z Thu Jan 1 00:00:00 1970 From: der.herr@hofr.at (Nicholas Mc Guire) Date: Fri, 13 Mar 2015 14:28:02 +0100 Subject: CHECK: Alignment should match open parenthesis In-Reply-To: <1158748566.6782126.1426252169494.JavaMail.yahoo@mail.yahoo.com> References: <20150313124332.GA22044@opentech.at> <1158748566.6782126.1426252169494.JavaMail.yahoo@mail.yahoo.com> Message-ID: <20150313132802.GA13910@opentech.at> To: kernelnewbies@lists.kernelnewbies.org List-Id: kernelnewbies.lists.kernelnewbies.org On Fri, 13 Mar 2015, palik imre wrote: > On Friday, 13 March 2015, 13:43, Nicholas Mc Guire wrote: > > On Fri, 13 Mar 2015, palik imre wrote: > > > > > > > Sorry for the silly question, but I have some issues with this checkpatch.pl warning. > > > > > > I mean Documentation/CodingStyle says: > > > > > > Outside of comments, documentation and except in Kconfig, spaces are never > > > used for indentation, and the above example is deliberately broken. > > > > > > But checkpatch.pl claims I should align to open parentheses. These two things seem to be contradictory to me. Could somebody clarify this? > > > > > > > leading tabs *followed* by spaces to align parameters to a function are fine > > The emacs settings in Documentation/CodingStyle seem to contradict to you, > as it is set up to use c-lineup-arglist-tabs-only > The problem is that CodingStyle does not explicitly address parameter alignment for functions that do not fit on a single line but checkpatch.pl does you can try it out - if you align to the opening braces with spaces with preceding TABs it will not fuss and this is also common practice. thx! hofrat