Search Linux Wireless

Re: [PATCH] Add iwlwifi wireless drivers

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

 



James Ketrenos wrote:
Randy Dunlap wrote:
15.  Limit source lines to <= 80 columns (this patch contains
over 200 lines that are > 80 columns).

Fixed in the .c files. Still have 155 instances in the .h files (most due to mid-line tabs)


See my email on this subject. This is NOT a hard and fast rule. If you have to do something stupid to the formatting to get it to fit, then you are moving in the wrong direction.

As an aside, sometimes long lines are an indication that a function needs to be broken up into smaller sub-functions.

	Jeff


-
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