Search Linux Wireless

Re: [PATCH 1/4] mac80211: Coding style cleanups

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

 



Johannes Berg wrote:
On Tue, 2007-03-20 at 10:39 +0000, andy@xxxxxxxxxxx wrote:

I don't really see why this is necessary at all, but anyway.

Yeah I cam to the same view -- I changed the file to match your suggestions here and realized that the (many) existing function defs do not follow your style recommendation. So I mass-changed them and it came to me I will only annoy the original author by this attempt at consistency. I started breaking out code into other functions and I realized I wasn't taking enough care, because it wasn't what I was trying to achieve, and would have to test it, might introduce bugs and would be run out of town by a mob with pitchforks.

I deleted the patch and learned to stop getting a tic in my eye every time a line > 80 cols flys by.

-Andy
-
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