Re: [PATCH] scsi: fix iscsi-related kernel-doc warnings

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

 



On 01/03/2018 08:11 PM, Martin K. Petersen wrote:
> 
> Randy,
> 
>> Fix kernel-doc warnings in drivers/scsi/ that are related to iscsi
>> support interfaces.
>>
>> Fixes these kernel-doc warnings: (tested by adding these files to a new
>> target.rst documentation file: WIP)
> 
> Applied to 4.16/scsi-queue, thanks!
> 
>> Cc:	"Nicholas A. Bellinger" <nab@xxxxxxxxxxxxxxx>
>> Cc:	linux-scsi@xxxxxxxxxxxxxxx
>> Cc:	target-devel@xxxxxxxxxxxxxxx
>> Cc:	Sagi Grimberg <sagi@xxxxxxxxxxx>
>> Cc:	linux-rdma@xxxxxxxxxxxxxxx
>> Cc:	"James E.J. Bottomley" <jejb@xxxxxxxxxxxxxxxxxx>
>> Cc:	"Martin K. Petersen" <martin.petersen@xxxxxxxxxx>
> 
>      ^^^ checkpatch complains about these spaces.

Hah, those are tabs, not spaces.

scripts/checkpatch.pl (comment) says something about this being a style thing:
# Check signature styles

WARNING: Use a single space after Cc:
#29: 
Cc:	"Martin K. Petersen" <martin.petersen@xxxxxxxxxx>



Email standards (RFCs) sure don't require a single space.

and I would like to be able to copy N lines from MAINTAINERS file and then
replace either the M: and/or L: (actually just the 'M' or 'L') with
"To" or "Cc" and be done with it, instead of having to replace TAB on each
line with a space also.

Does anyone know where this "signature style" check came from?
I don't see it in any style guide.

thanks,
-- 
~Randy
--
To unsubscribe from this list: send the line "unsubscribe linux-rdma" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html



[Index of Archives]     [Linux USB Devel]     [Video for Linux]     [Linux Audio Users]     [Photo]     [Yosemite News]     [Yosemite Photos]     [Linux Kernel]     [Linux SCSI]     [XFree86]
  Powered by Linux