Re: connector.c

[Matthias Urlichs]

Hi,   Tommy Reynolds schrub am Thu, 31 Mar 2005 20:41:35 -0600:

> Uttered Andrew Morton <akpm@xxxxxxxx>, spake thus:
> 
>> > 	if (uskb) {
>> > 		netlink_unicast(dev->nls, uskb, 0, 0);
>> > 	}
>> 
>> Unneeded {}
> 
> However, for maintainability (and best practices) they are essential.

They do add visual clutter, though, so they make the code as-is less
readable.

I don't think it's entirely accidental that Python is so much more
readable. (For me, anyway -- YMMV and all that.)

-- 
Matthias Urlichs   |   {M:U} IT Design @ m-u-it.de   |  smurf@xxxxxxxxxxxxxx


-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/