[webkit-dev] Coding style

[Kenneth Christiansen]

OK, done! :-)

https://bugs.webkit.org/show_bug.cgi?id=26188

Cheers,
Kenneth

On Thu, Jun 4, 2009 at 12:13 PM, David Kilzer <ddkilzer at webkit.org> 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 <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
>