Re: [PATCH] drivers: Conversions from kmalloc+memset to k(z|c)alloc.
From: Michael Buesch
Date: Sun Jul 23 2006 - 14:23:04 EST
On Sunday 23 July 2006 19:55, Tomasz KÅoczko wrote:
> On Sun, 23 Jul 2006, Alexey Dobriyan wrote:
> >> Again: using indent mainly will mean only one time massive changes.
> > True, 180M(!) of them.
> And this is so huge now because seems there is no obligation use common
> format .. all is formated using hands/mind/difftent editors autoformaters.
> >> After
> >> this ident can be runed for example by Linus just before make release
> >> and/or partial release.
> > ~4M per run.
> If patch submmitter will use formating tool and it will add it will statr
> work on formated source tree it will be 0M per run.
> >>> scripts/Lindent exists and gets used, but it is not perfect.
> > Correction: GNU indent exists and gets used, but it is not perfect.
> Yes .. and produce by Lindent ~160MB patch it excelent proof how offent is
> is used now :>
> (please stop this crap "argumentation" :>)
Yeah, please stop it.
Did you actually _look_ at what indent does to code sometimes?
It sometimes (often?) renders perfectly readable code into a
huge blob of crap.
Face reality. The linux kernel is following the general codingstyle
very well already. I don't think there is need to improve the current
codebase for non-existent codingstyle issues. And we already review
new code for codingstyle issues, so the codebase remains clean.
Look at other projects with horrible codingstyle problems
and suggest solutions to their _real_ issues. *cough*kde*cough*
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/