Re: CHECK: Alignment should match open parenthesis

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Mar 13, 2015 at 3:47 PM, Nicholas Mc Guire <der.herr@xxxxxxx> wrote:
> On Fri, 13 Mar 2015, Nicholas Mc Guire wrote:
>
>> On Fri, 13 Mar 2015, palik imre wrote:
>>
>> > On Friday, 13 March 2015, 13:43, Nicholas Mc Guire <der.herr@xxxxxxx> 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.
>>
> here is a quick shot at summarizing this
>
>
> If the parameter list to a functions would exceed the 80 char limit then break
> it at the separators, and align to opening braces, e.g.:
>
>                         ret = fw_load_from_user_helper(fw, name, device,
>                                                        opt_flags, timeout);
>
> or:
>
>         int =
>         wait_for_completion_interruptible_timeout(data->completion,
>                                                   PMI_TIMEOUT);
>
> Note that this is indented by tabs and then aligned with spaced to fit the
> opening braces. If you can not fit it even if you break the parameter list
> at the commas then indent by tabs only but *significantly* to the left of
> the opening braces, e.g.:
>
>                 int ret = wait_for_completion_interruptible_timeout(
>                                 &info->done, usecs_to_jiffies(TIMEOUT_US));
>
>
> would be suprised if there is no writeup somewhere alredy
> but I did not find this covered in Documentations anywhere.

I think it would be a coding idea to have this in CodingStyle doc :).

Daniel.

_______________________________________________
Kernelnewbies mailing list
Kernelnewbies@xxxxxxxxxxxxxxxxx
http://lists.kernelnewbies.org/mailman/listinfo/kernelnewbies




[Index of Archives]     [Newbies FAQ]     [Linux Kernel Mentors]     [Linux Kernel Development]     [IETF Annouce]     [Git]     [Networking]     [Security]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux RAID]     [Linux SCSI]     [Linux ACPI]
  Powered by Linux