----- Original Message ---- > From: Joe Perches <joe@xxxxxxxxxxx> > To: Krzysztof Halasa <khc@xxxxxxxxx> > Cc: Samuel Thibault <samuel.thibault@xxxxxxxxxxxx>; J. Bruce Fields ><bfields@xxxxxxxxxxxx>; Dan Carpenter <error27@xxxxxxxxx>; Andy Whitcroft ><apw@xxxxxxxxxxxxx>; linux-kernel@xxxxxxxxxxxxxxx; >kernel-janitors@xxxxxxxxxxxxxxx > Sent: Wed, January 5, 2011 6:45:29 PM > Subject: Re: [patch] checkpatch: putting the && or || on the wrong line > > > Increasing e.g. > > readability of the code would be a good reason, but statistics? > > Standardization of style in a large project increases > readability and decreases errors full stop. Anything that prevents errors from happening is good. No question. Anything that enforces "style" for no good reason is just counterproductive "readability" is a personal thing. It cannot be increased by enforcing "style". Not for everybody. And when discussing this issue we should not forget the environment. If I am a programmer working for a big$$$ company, I better follow their style-guide (if they have one). They pay my food and shelter after all. If I am "just" a volunteer on some mainly volunteer open source project, why should I care about "in my opinion" crap style. It will just drive me away from that project. Again as I wrote elsewhere - all depends on context. If a contribution is just a small patch to existing work one can easily follow the "style" of the existing work. If it is genuinely new work, it should be valued by the peers on technical merit, not on whether they like the contributors way of writing out an condition. > > Outside of that, it's _all_ bikeshedding. Your shed is my castle > > cheers, Joe Cheers Martrin (to hell with style) -- To unsubscribe from this list: send the line "unsubscribe kernel-janitors" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html