[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]
Message-ID: <45BD4D2C.2010306@student.ltu.se>
Date: Mon, 29 Jan 2007 02:26:04 +0100
From: Richard Knutsson <ricknu-0@...dent.ltu.se>
To: Oleg Verych <olecom@...wer.upol.cz>, Andrew Morton <akpm@...l.org>
CC: LKML <linux-kernel@...r.kernel.org>
Subject: Re: [SCRIPT] Remove "space damage" from patches
Oleg Verych wrote:
> 29-01-2007, Richard Knutsson:
>
>> Oleg Verych wrote:
>>
>>> Nack, big and fat. It's userspace problem.
>>>
>>> If our "developers" can't use or update their every-day-tools, it's a
>>> shame. And this is not trolling: i'm using Jed and GNU Emacs, both are
>>> ok with whitespace "production".
>>>
>> Oh, I hope I didn't give the impression I wanted it in the kernel (that
>> is why i labeled it as SCRIPT and not PATCH), as you said it is a
>> userspace problem. I just thought a simple script to remove those
>> whitespace could help in an imperfect world. I prefer kate since then
>> you can see where the tabs begins (and other features).
>>
>
> Just to give you idea, how imperfect it is:
> <http://marc.theaimsgroup.com/?l=linux-mm-commits&m=116198944205036&w=2>
>
> Anyway, i still think programmers *must* take care of it, if they think
> they _are_ Programmers. IMHO LKML already gives obligation to be one
Then I hope you don't mind me asking, why is there (L)indent? Everyone
can make a mistake and if your editor does not auto-format then there
may be a whitespace straying.
I also guess you saw that the script is _not_ for cleaning up
source-files (can be intrusive and is better to be fixed when fixing
something else), but patches. So if a maintainer does not have anything
to easily fix those, they might want a simple script to do the work
(otherwise I believe they just deleted my mail ;) ).
As I said in the posting, it is just if someone is interested in it,
otherwise you have a Del-button.
Richard Knutsson
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists