Coding style: do_this(a,b) vs. do_this(a, b)
From: Pavel Machek (pavel_at_ucw.cz)
Date: 08/17/04
- Previous message: Daniel Paschka: "Problem with CIFS"
- Next in thread: Albert Cahalan: "Re: Coding style: do_this(a,b) vs. do_this(a, b)"
- Maybe reply: Albert Cahalan: "Re: Coding style: do_this(a,b) vs. do_this(a, b)"
- Reply: Fao, Sean: "Re: Coding style: do_this(a,b) vs. do_this(a, b)"
- Reply: Frank van Maarseveen: "Re: Coding style: do_this(a,b) vs. do_this(a, b)"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Date: Tue, 17 Aug 2004 12:38:52 +0200 To: kernel list <linux-kernel@vger.kernel.org>, Andrew Morton <akpm@zip.com.au>
Hi!
Coding style document is not consistent with itself on whether there
should be space after ","... This makes it standartize on ", " option.
Pavel
--- tmp/linux/Documentation/CodingStyle 2004-05-20 23:08:01.000000000 +0200
+++ linux/Documentation/CodingStyle 2004-06-06 00:27:11.000000000 +0200
@@ -356,11 +356,11 @@
Macros with multiple statements should be enclosed in a do - while block:
-#define macrofun(a,b,c) \
+#define macrofun(a, b, c) \
do { \
if (a == 5) \
- do_this(b,c); \
- } while (0)
+ do_this(b, c); \
+ } while(0)
Things to avoid when using macros:
-- People were complaining that M$ turns users into beta-testers... ...jr ghea gurz vagb qrirybcref, naq gurl frrz gb yvxr vg gung jnl! - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
- Previous message: Daniel Paschka: "Problem with CIFS"
- Next in thread: Albert Cahalan: "Re: Coding style: do_this(a,b) vs. do_this(a, b)"
- Maybe reply: Albert Cahalan: "Re: Coding style: do_this(a,b) vs. do_this(a, b)"
- Reply: Fao, Sean: "Re: Coding style: do_this(a,b) vs. do_this(a, b)"
- Reply: Frank van Maarseveen: "Re: Coding style: do_this(a,b) vs. do_this(a, b)"
- Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]
Relevant Pages
|
|