Re: CodingStyle: start flamewar about use of braces

2007-05-09 Thread Pekka Enberg
On 5/9/07, Stefan Richter <[EMAIL PROTECTED]> wrote: I don't know which form is better suited to the goal of well maintainable code --- the compact, easy to read, winning guideline, or the comprehensive norm. Bah, it's a slippery slope. Too little guidelines and we end up people submitting code

Re: CodingStyle: start flamewar about use of braces

2007-05-09 Thread Stefan Richter
Satyam Sharma wrote: > It's quite funny to worship them as "prophets" and adopt their style > for some stuff (with a simple "Rationale: K&R.") one instant and then > suggest a style completely opposite to theirs one screenful later in > the same file. That's what happens to a text which is increme

Re: CodingStyle: start flamewar about use of braces

2007-05-08 Thread Satyam Sharma
PROTECTED]> >> CommitDate: Tue May 8 11:15:12 2007 -0700 >> >> CodingStyle: start flamewar about use of braces > > It worked somewhat. We never did reach any kind of > concensus about this > > >> Signed-off-by: Oliver Neukum <[EMAIL PROTECTED]> >>

Re: CodingStyle: start flamewar about use of braces

2007-05-08 Thread Jeff Garzik
Stephen Clark wrote: Lennart Sorensen wrote: On Tue, May 08, 2007 at 05:19:45PM -0400, Jeff Garzik wrote: If anyone tries to add braces to my code's 'else' statements where they are not required, that patch will get NAK'd in a heartbeat. Oh isn't coding style fun. I personally hate code tha

Re: CodingStyle: start flamewar about use of braces

2007-05-08 Thread Stephen Clark
Lennart Sorensen wrote: On Tue, May 08, 2007 at 05:19:45PM -0400, Jeff Garzik wrote: +Do not unnecessarily use braces where a single statement will do. + +if (condition) + action(); + +This does not apply if one branch of a conditional statement is a single +statement. Use braces in bo

Re: CodingStyle: start flamewar about use of braces

2007-05-08 Thread Lennart Sorensen
On Tue, May 08, 2007 at 05:19:45PM -0400, Jeff Garzik wrote: > >>+Do not unnecessarily use braces where a single statement will do. > >>+ > >>+if (condition) > >>+ action(); > >>+ > >>+This does not apply if one branch of a conditional statement is a single > >>+statement. Use braces in both bran

Re: CodingStyle: start flamewar about use of braces

2007-05-08 Thread Jeff Garzik
: 28be5abb400e5e082f5225105fdc69337ec0c0b4 Author: Oliver Neukum <[EMAIL PROTECTED]> AuthorDate: Tue May 8 00:30:34 2007 -0700 Committer: Linus Torvalds <[EMAIL PROTECTED]> CommitDate: Tue May 8 11:15:12 2007 -0700 CodingStyle: start flamewar about use of braces It worked somewhat.

Re: CodingStyle: start flamewar about use of braces

2007-05-08 Thread Randy Dunlap
> Parent: 28be5abb400e5e082f5225105fdc69337ec0c0b4 > Author: Oliver Neukum <[EMAIL PROTECTED]> > AuthorDate: Tue May 8 00:30:34 2007 -0700 > Committer: Linus Torvalds <[EMAIL PROTECTED]> > CommitDate: Tue May 8 11:15:12 2007 -0700 > > CodingStyle: start flamewar about use of brace