Search Linux Wireless

Re: [PATCH 20/20] staging: brcm80211: reduced checkpatch warnings to zero

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

 



On 09/19/2011 05:04 PM, Joe Perches wrote:
Splitting format strings is poor style, makes
error messages difficult to find via grep and
is error prone.

You should update your checkpatch version.
Current checkpatch doesn't warn about this.

Thank you for pointing this out.

It'd be better if you could find some way
to reduce the 6 tab indentation rather than
split 20 char format strings into pieces.

Greg, could you please drop this one? We will take care of the indentation issue in future patches.

Thanks,
Franky

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


[Index of Archives]     [Linux Host AP]     [ATH6KL]     [Linux Bluetooth]     [Linux Netdev]     [Kernel Newbies]     [Linux Kernel]     [IDE]     [Security]     [Git]     [Netfilter]     [Bugtraq]     [Yosemite News]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux ATA RAID]     [Samba]     [Device Mapper]
  Powered by Linux