Re: [PATCH v4] checkpatch: debugfs_remove() can take NULL

[Andy Whitcroft]

On Tue, Nov 20, 2012 at 05:10:43PM +0200, Constantine Shulyupin wrote:
> On Tue, Nov 20, 2012 at 4:58 PM, Joe Perches <joe@xxxxxxxxxxx> wrote:
> > I don't think it's an issue.  I'm not sure anyone
> > really uses --ignore for much other than LONG_LINE.
> 
> Indeed. Must all code be 80 characters width?  My be my be better to
> make line length configurable than ignore it? Or at least print actual
> line length to allow developer be aware how long lines is.
> I think 90 char line is preferable than wrapped two or even three times line.

If wrapping there is that much better then wrapping there is appropriate
in some cases.  The currently limit has been revisited a bunch of times,
and in none have we agreed on what to change it to.  It therefore
remains the recommendation.

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