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:

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

diff --git a/WebKitSite/ChangeLog b/WebKitSite/ChangeLog
index 69812167188b91bfaea2942bbd93d5915ec5e640..4d4867d51bfdea3bb64cebeec92ede51e42d629f 100644 (file)
--- a/WebKitSite/ChangeLog
+++ b/WebKitSite/ChangeLog
@@ -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

diff --git a/WebKitSite/coding/coding-style.html b/WebKitSite/coding/coding-style.html
index cd682dc6989219e4240c80e5f6bf048461284a8e..776206c582342bfbfefd6a3d31061f082c439d52 100644 (file)
--- a/WebKitSite/coding/coding-style.html
+++ b/WebKitSite/coding/coding-style.html
@@ -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>