Re: [PATCH] kernel/module.c has something to hide. (whitespace cleanup)

[Nick Piggin]

Bodo Eggert <harvested.in.lkml@xxxxxxxxxxxxxxxxxxxxxxxxxx> wrote:

> Jesper Juhl <juhl-lkml@xxxxxx> wrote:
>
>
> > If Andrew agrees, then I'll commit to doing this cleanup;
>
> > - (to a limited degree) no trailing whitespace
>
> I just ran a script over -rc4 to remove trailing ws. The result is
> about 22 MB in 429 patches (iterated over ./*/*).
>
> How hard can I patch you before you start patching me?
>
> Which addresses am I supposed to send it to? I don't want to break the
> record for the most annoying patch series in lkml.

First of all, why is it 429 patches? The patches we want aren't about a
file or a subdirectory or even a subsystem, but they're supposed to be
a logical change. Ie. 1 patch. An exception for something like this would
be if you want to feed it to different maintainers seperately, but it
sounds like you just want to bomb it somewhere...

Secondly, let's not.

More subtle indenting stuff like fixing `if (exp) do_something` I can
understand. But this isn't nearly so helpful.


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