<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.0 Transitional//EN">
<HTML><HEAD>
<META content="text/html; charset=iso-8859-1" http-equiv=Content-Type>
<META name=GENERATOR content="MSHTML 8.00.6001.18813">
<STYLE></STYLE>
</HEAD>
<BODY bgColor=#ffffff>
<DIV><PRE class=code>Current guideline also contains these 2 cases that {} should be used. I think when the condition is multi-lined, this should also apply. </PRE><PRE class=code>(BTW, hate the existing rule "no braces for one line". it doesn't give any benefit. worse than "always use braces")</PRE><PRE class=code>if (condition) {
    // Some comment
    doIt();
}

if (condition) {
    myFunction(reallyLongParam1, reallyLongParam2, ...
        reallyLongParam5);
}
</PRE></DIV>
<BLOCKQUOTE 
style="BORDER-LEFT: #000000 2px solid; PADDING-LEFT: 5px; PADDING-RIGHT: 0px; MARGIN-LEFT: 5px; MARGIN-RIGHT: 0px" 
dir=ltr>
  <DIV style="FONT: 10pt arial">----- Original Message ----- </DIV>
  <DIV 
  style="FONT: 10pt arial; BACKGROUND: #e4e4e4; font-color: black"><B>From:</B> 
  <A title=levin@chromium.org href="mailto:levin@chromium.org">David Levin</A> 
  </DIV>
  <DIV style="FONT: 10pt arial"><B>To:</B> <A title=yong.li@torchmobile.com 
  href="mailto:yong.li@torchmobile.com">Yong Li</A> </DIV>
  <DIV style="FONT: 10pt arial"><B>Cc:</B> <A title=webkit-dev@lists.webkit.org 
  href="mailto:webkit-dev@lists.webkit.org">WebKit Development</A> </DIV>
  <DIV style="FONT: 10pt arial"><B>Sent:</B> Wednesday, September 02, 2009 2:56 
  PM</DIV>
  <DIV style="FONT: 10pt arial"><B>Subject:</B> Re: [webkit-dev] unwritten rules 
  of webkit style</DIV>
  <DIV><BR></DIV><BR><BR>
  <DIV class=gmail_quote>On Wed, Sep 2, 2009 at 11:54 AM, Yong Li <SPAN 
  dir=ltr>&lt;<A href="mailto:yong.li@torchmobile.com">yong.li@<A 
  href="http://torchmobile.com">torchmobile.com</A></A>&gt;</SPAN> wrote:<BR>
  <BLOCKQUOTE 
  style="BORDER-LEFT: #ccc 1px solid; MARGIN: 0px 0px 0px 0.8ex; PADDING-LEFT: 1ex" 
  class=gmail_quote>
    <DIV style="WORD-WRAP: break-word" bgcolor="#ffffff">
    <DIV align=left><FONT size=2 face=Arial>{} should be added in this 
    case:</FONT></DIV>
    <DIV align=left><FONT size=2 face=Arial></FONT>&nbsp;</DIV>
    <BLOCKQUOTE 
    style="BORDER-BOTTOM: medium none; BORDER-LEFT: medium none; PADDING-BOTTOM: 0px; MARGIN: 0px 0px 0px 40px; PADDING-LEFT: 0px; PADDING-RIGHT: 0px; BORDER-TOP: medium none; BORDER-RIGHT: medium none; PADDING-TOP: 0px"><FONT 
      face="'courier new', monospace">if (condition1</FONT> 
      <DIV><FONT face="'courier new', monospace">&nbsp;&nbsp; &nbsp; &nbsp; 
      &nbsp;&amp;&amp; condition2)</FONT></DIV>
      <DIV><FONT face="'courier new', monospace">&nbsp;&nbsp; 
      &nbsp;statement;</FONT></DIV>
      <DIV align=left><FONT 
  face="Courier New"></FONT></DIV></BLOCKQUOTE></DIV></BLOCKQUOTE>
  <DIV><BR></DIV>
  <DIV>Not according to current WebKit style because it is a single line 
  statement.</DIV>
  <DIV><BR></DIV></DIV></BLOCKQUOTE></BODY></HTML>