Re: [patch] checkpatch: relax spacing and line length

From: Boaz Harrosh
Date: Mon Apr 07 2008 - 05:53:34 EST


On Sun, Apr 06 2008 at 21:52 +0300, Joe Perches <joe@xxxxxxxxxxx> wrote:
> On Sun, 2008-04-06 at 14:52 +0300, Benny Halevy wrote:
>> The idea is [] to allow for a bit more flexibility to
>> accommodate for a more robust indentation style.
>
> Isn't this an editor/tools issue more than a checkpatch one?
>
> I think you should first try to get the most frequently used
> code editors and pretty printers (vim/emacs/eclipse/indent, etc)
> to support the tabs to level, spaces to align style.
>
> g'luck with that.
>
> --

I have one such editor to date, Me. But I can not use it's output
because checkpatch.pl forbids it, and I get scorned by my maintainers.
So I think you got it reversed. First allow it then automate for it
other wise there is no use, right?

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