Re: [net-next-2.6 04/15] e1000e: checkpatch error - open braces

[Krzysztof Halasa <khc@pm.waw.pl>]

Jeff Garzik <jeff@garzik.org> writes:

>> ERROR: that open brace { should be on the previous line

>> -static const u16 e1000_gg82563_cable_length_table[] =
>> -	 { 0, 60, 115, 150, 150, 60, 115, 150, 180, 180, 0xFF };

>> -	static const u32 test[] =
>> -		{0x5A5A5A5A, 0xA5A5A5A5, 0x00000000, 0xFFFFFFFF};

> Gah, that looks worse than the unpatched code.
>
> I think this was an over-literal checkpatch interpretation.  If you
> look at e.g. drivers/net/tg3.c, you see that longer tables have the
> trailing braces on a separate line, where they should be.
>
> IMO, the single-line cases should just be left as they are now in the
> source code.

I think so. Also I think it's a false positive - the above aren't the
typical
	if()
		{
		xxx
		}

They are instead single lines:
	type var[] = {a, b, c, d};
broken into two to meet the (former) line length limit.

Makes me think that we should never break long lines in the code. This
is best left for the text editors and viewers (only while displaying).
-- 
Krzysztof Halasa