Re: [PATCH 13/13] usb: core: config: fix parenthesis alignment

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

 



On Tue, Oct 05, 2021 at 02:52:43PM +0100, Robert Greener wrote:
> Fixes the following checkpatch.pl warnings at various locations:
> 
> CHECK:PARENTHESIS_ALIGNMENT: Alignment should match open parenthesis

This is yet another case where I disagree with checkpatch.pl.  C isn't
Lisp; we shouldn't have to use the same conventions about aligning
parentheses.

My preference is to indent continuation lines by two tab stops.  In
config.c I modified this practice because the continuation lines were
very long and were indented very far; instead I sometimes indented
them by four spaces.  But in no case does it seem reasonable to align
continuation lines with opening parens.

Alan Stern



[Index of Archives]     [Linux Media]     [Linux Input]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]     [Old Linux USB Devel Archive]

  Powered by Linux