Re: [dm-devel] Re: [PATCH] scripts/checkpatch.pl: Change long linewarning to 105 chars

From: Alasdair G Kergon
Date: Fri Dec 18 2009 - 10:13:23 EST


On Fri, Dec 18, 2009 at 03:37:00PM +0100, Krzysztof Halasa wrote:
> Not sure if that's long enough (maybe it is).
> 132 looks like the next "natural" number.

Well it's good that this thread has produced more examples where it's
reasonable and acceptable to exceed 80 characters.

What do people feel about files where the policy is to place all the
parameters passed into a function on the same line, regardless of its
consequent length?

(What kicked this all off was a patch Mikulas submitted containing many
long lines, one of which hits column 264. Personally, I dislike reading
code with lines that wrap, but using a 132-column terminal width is
fine.)

Alasdair

--
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/