2011-04-27 Ojan Vafai <ojan@chromium.org>
authorojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 27 Apr 2011 18:35:50 +0000 (18:35 +0000)
committerojan@chromium.org <ojan@chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 27 Apr 2011 18:35:50 +0000 (18:35 +0000)
        Reviewed by Dan Bernstein.

        change style guide to pre-increment in for-loops
        https://bugs.webkit.org/show_bug.cgi?id=59619

        This matches what a number of reviewers have been asking
        for in code reviews.

        * coding/coding-style.html:

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

Websites/webkit.org/ChangeLog
Websites/webkit.org/coding/coding-style.html

index 0abe4034267826fdecbe607626e86aa3035ca892..5f862a04b36d269b5c22f666711b2c2f72ccb157 100644 (file)
@@ -1,3 +1,15 @@
+2011-04-27  Ojan Vafai  <ojan@chromium.org>
+
+        Reviewed by Dan Bernstein.
+
+        change style guide to pre-increment in for-loops
+        https://bugs.webkit.org/show_bug.cgi?id=59619
+
+        This matches what a number of reviewers have been asking
+        for in code reviews.
+
+        * coding/coding-style.html:
+
 2011-04-26  Carol Szabo  <carol.szabo@nokia.com>
 
         Reviewed by Tor Arne Vestbø.
index 4331aaf1212289c8cb03454216a6f9d46a911299..307cfddd4300a955608389da7c30a7c81e24e591 100644 (file)
@@ -185,7 +185,7 @@ return condition ? 1:0;
 <li>Do not place spaces before comma and semicolon.
 <h4 class="right">Right:</h4>
 <pre class="code">
-for (int i = 0; i &lt; 10; i++)
+for (int i = 0; i &lt; 10; ++i)
     doSomething();
 
 f(a, b);
@@ -193,7 +193,7 @@ f(a, b);
 
 <h4 class="wrong">Wrong:</h4>
 <pre class="code">
-for (int i = 0 ; i &lt; 10 ; i++)
+for (int i = 0 ; i &lt; 10 ; ++i)
     doSomething();
 
 f(a , b) ;
@@ -340,7 +340,7 @@ namespace WebCore {
     ...
 }
 
-for (int i = 0; i &lt; 10; i++) {
+for (int i = 0; i &lt; 10; ++i) {
     ...
 }
 </pre>