Re: [PATCH] drivers: Conversions from kmalloc+memset to k(z|c)alloc.

From: Tomasz Kłoczko
Date: Sat Jul 22 2006 - 16:58:52 EST


On Sat, 22 Jul 2006, Jeff Garzik wrote:

Tomasz Kłoczko wrote:
Moment .. are you want to say something like "keep commont coding style can't be maintained by tool" ?
Even if indent watches on to small coding style emenets still I don't see why using this tool isn't one of the current ement of release procedure (?).

indent isn't perfect, _especially_ where C99 comes into the picture.

Again: is in this case "isn't perfect" mean "it does not make all what we want" ? If yes still I don't see why not use indent + some other tool or if you will show real example where it does something badady (like now for checking code syntax is used compiler and some other tools like sparse).

And running indent across the tree pre-release would (a) create a ton of noise before each release, and (b) undo perfectly valid, readable formatting.

Committing all this "noise" will plug all this thing and allow reve most of content Documentation/CodingStyle document.
Is it not wort stop all questions/discuss/flames on this subject ?
Again: using indent mainly will mean only one time massive changes. After this ident can be runed for example by Linus just before make release and/or partial release.

scripts/Lindent exists and gets used, but it is not perfect.

Again: anywhere are listed/was posted list of "not perfect" examples ? And/or: what does it mean in this case "not perfect" ? Show this for allow start work on fix indent by other people (if all cases will be resul of some bugs in this tool).

kloczek
--
-----------------------------------------------------------
*Ludzie nie mają problemów, tylko sobie sami je stwarzają*
-----------------------------------------------------------
Tomasz Kłoczko, sys adm @zie.pg.gda.pl|*e-mail: kloczek@xxxxxxxxxxxxxxxxxx*