2007-11-21 Mark Rowe <mrowe@apple.com>
authormrowe@apple.com <mrowe@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 22 Nov 2007 04:13:22 +0000 (04:13 +0000)
committermrowe@apple.com <mrowe@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 22 Nov 2007 04:13:22 +0000 (04:13 +0000)
        Reviewed by Eric.

        Update coding style to cover control statements with no body.

        * coding/coding-style.html:

git-svn-id: https://svn.webkit.org/repository/webkit/trunk@27950 268f45cc-cd09-0410-ab3c-d52691b4dbfc

WebKitSite/ChangeLog
WebKitSite/coding/coding-style.html

index 6981216..4d4867d 100644 (file)
@@ -1,3 +1,11 @@
+2007-11-21  Mark Rowe  <mrowe@apple.com>
+
+        Reviewed by Eric.
+
+        Update coding style to cover control statements with no body.
+
+        * coding/coding-style.html:
+
 2007-11-17  Adam Roben  <aroben@apple.com>
 
         Update Windows debugging instructions
index cd682dc..776206c 100644 (file)
@@ -280,6 +280,18 @@ if (condition) {
 }
 </pre>
 </li>
+
+<li>Control clauses without a body should use empty braces:
+<h4 class="right">Right:</h4>
+<pre class="code">
+for ( ; current; current = current->next) { }
+</pre>
+
+<h4 class="wrong">Wrong:</h4>
+<pre class="code">
+for ( ; current; current = current->next);
+</pre>
+</li>
 </ol>
 
 <h3>Null, false and 0</h3>