<!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><FONT size=2 face=Arial>For multi-line condition, the following style is 
most readable to me.</FONT></DIV>
<DIV><FONT size=2 face=Arial></FONT>&nbsp;</DIV>
<DIV><FONT size=2 face=Arial>if (condition1 </FONT></DIV>
<DIV><FONT size=2 face=Arial>&nbsp;&nbsp;&nbsp; &amp;&amp; 
condition2)</FONT></DIV>
<DIV><FONT size=2 face=Arial>{</FONT></DIV>
<DIV><FONT size=2 face=Arial>&nbsp;&nbsp;&nbsp;&nbsp; // code...</FONT></DIV>
<DIV><FONT size=2 face=Arial>}</FONT></DIV>
<DIV><FONT size=2 face=Arial></FONT>&nbsp;</DIV>
<BLOCKQUOTE 
style="BORDER-LEFT: #000000 2px solid; PADDING-LEFT: 5px; PADDING-RIGHT: 0px; MARGIN-LEFT: 5px; MARGIN-RIGHT: 0px">
  <DIV style="FONT: 10pt arial">----- Original Message ----- </DIV>
  <DIV 
  style="FONT: 10pt arial; BACKGROUND: #e4e4e4; font-color: black"><B>From:</B> 
  <A title=yong.li@torchmobile.com href="mailto:yong.li@torchmobile.com">Yong 
  Li</A> </DIV>
  <DIV style="FONT: 10pt arial"><B>To:</B> <A title=levin@chromium.org 
  href="mailto:levin@chromium.org">David Levin</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 3:05 
  PM</DIV>
  <DIV style="FONT: 10pt arial"><B>Subject:</B> Re: [webkit-dev] unwritten rules 
  of webkit style</DIV>
  <DIV><BR></DIV>
  <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>
  <P>
  <HR>

  <P></P>_______________________________________________<BR>webkit-dev mailing 
  list<BR>webkit-dev@lists.webkit.org<BR>http://lists.webkit.org/mailman/listinfo.cgi/webkit-dev<BR></BLOCKQUOTE></BODY></HTML>