[webkit-dev] Coding style

[David Kilzer]

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 <kenneth.christiansen at openbossa.org>
To: webkit-dev <webkit-dev at lists.webkit.org>
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
webkit-dev at lists.webkit.org
http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20090604/083306e6/attachment.html>