Thread (6 messages) flat view 6 messages, 5 authors, 2012-02-20

RE: [PATCH] Cleaning up code formatting errors in net/wireless pointed out by checkpatch.

From: David Laight <hidden>
Date: 2012-02-17 15:48:14
Also in: linux-wireless, lkml

 
These are mostly minor changes and they are being sent as one 
patch only in order to cause the smallest amount of disruption.
A lot of these changes don't seem to match what I thought the
linux kernel code style is.

eg:
 void cfg80211_bss_age(struct cfg80211_registered_device *dev,
-                      unsigned long age_secs);
+		      unsigned long age_secs);
Args no longer lined up.
-		pos = (u8 *) & hdr->addr4;
+		pos = (u8 *) &hdr->addr4;
no space after cast?
-	p1 = (u8*)(ht_capa);
+	p1 = (u8 *)(ht_capa);
Still extra ().

I'm also not sure (personally) this sort of churn is useful.

	David
Keyboard shortcuts
hback out one level
jnext message in thread
kprevious message in thread
ldrill in
Escclose help / fold thread tree
?toggle this help