On Wed, 21 Dec 2011 06:58:19 +0100, Joe Perches <joe@xxxxxxxxxxx> wrote:
On Wed, 2011-12-21 at 14:45 +0900, Minchan Kim wrote:
I feel it's no problem as it is because it's very short function now
but it's not style we prefer.
Who is this "we" you refer to?
There's nothing suggesting your patch as a preferred style
in Documentation/CodingStyle.
There is plenty that is not documented, ;) yet it seems the community prefers
having ifdefs outside of the function.
--
Best regards, _ _
.o. | Liege of Serenely Enlightened Majesty of o' \,=./ `o
..o | Computer Science, Michał “mina86” Nazarewicz (o o)
ooo +----<email/xmpp: mpn@xxxxxxxxxx>--------------ooO--(_)--Ooo--
--
To unsubscribe, send a message with 'unsubscribe linux-mm' in
the body to majordomo@xxxxxxxxx. For more info on Linux MM,
see: http://www.linux-mm.org/ .
Fight unfair telecom internet charges in Canada: sign http://stopthemeter.ca/
Don't email: <a href