lists.openwall.net | lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC | |
Open Source and information security mailing list archives
| ||
|
Date: Fri, 4 May 2007 23:16:55 +0200 (MEST) From: Jan Engelhardt <jengelh@...ux01.gwdg.de> To: Jiri Slaby <jirislaby@...il.com> cc: Oliver Neukum <oliver@...kum.org>, linux-kernel@...r.kernel.org Subject: Re: [patch]clarification of coding style regarding conditional statements with two branches On May 4 2007 16:48, Jiri Slaby 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 branches. > >Why, what's wrong with >if (condition) { > do_this(); > do_that(); >} else > otherwise(); > >? It's more readable/nicer in my eyes. I'll just point to an opinion: http://lkml.org/lkml/2006/9/5/36 , lower half. ("Secondly, whenever...") Jan -- - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@...r.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
Powered by blists - more mailing lists