OK, done! :-) https://bugs.webkit.org/show_bug.cgi?id=26188
Cheers, Kenneth On Thu, Jun 4, 2009 at 12:13 PM, David Kilzer <[email protected]> wrote: > Hi Kenneth, > The examples you give below are correct (with the exception of the number of > spaces used to indent the code in the else blocks). > Please file a bug on bugs.webkit.org and attach a patch. The webkit.org web > site is in the svn repository, so you may create patches against the HTML. > Thanks! > Dave > > ________________________________ > From: Kenneth Christiansen <[email protected]> > To: webkit-dev <[email protected]> > Sent: Thursday, June 4, 2009 7:52:29 AM > Subject: [webkit-dev] Coding style > > Hi there, > > I'm having a question regarding the coding style. > > According to 2. An else statement should go on the same line as a > preceding close brace. > > I would always need a brace when using if-else, in order to but the > else statement on the same line as the preceding close brace, is this > right? > or would something like this is OK: > > if (condition) > func1(); > else > func2(); > > And what about this case > > if (condition) > func1(); > else { > func2(); > func3(); > } > > It would be nice to have this defined in the coding style, as well as > adding the case that when you have a comment inside a one line if > statement you will need braces, like > > if (condition) { > // comment > func() > } > > Cheers, Kenneth > _______________________________________________ > webkit-dev mailing list > [email protected] > http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev > _______________________________________________ webkit-dev mailing list [email protected] http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
