Reviewed by Adel.
authorkmccullo <kmccullo@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 9 Sep 2006 00:11:01 +0000 (00:11 +0000)
committerkmccullo <kmccullo@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Sat, 9 Sep 2006 00:11:01 +0000 (00:11 +0000)
        - this patch brought to you by the letter 'c'

        * coding/coding-style.html:

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

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

index 3fe7576504a56fea77cbf827475801e07615dbec..52c988137cb4de19fdfcf5549bdb44ffb9a80196 100644 (file)
@@ -1,3 +1,11 @@
+2006-09-08  Kevin McCullough  <KMcCullough@apple.com>
+
+        Reviewed by Adel.
+
+        - this patch brought to you by the letter 'c'
+
+        * coding/coding-style.html:
+
 2006-08-24  Mitz Pettel  <opendarwin.org@mitzpettel.com>
 
         Reviewed by Darin.
index b509bf2e7efe2cb3df190527424e6bdfea6a5c08..6b0e1f1deb180fe47df01fe18b67c15d1d37267f 100644 (file)
@@ -1,6 +1,6 @@
 <?php 
-       $title="WebKit Coding Style Guidelines";
-       include("../header.inc"); 
+    $title="WebKit Coding Style Guidelines";
+    include("../header.inc"); 
 ?>
 
 
@@ -75,16 +75,16 @@ if (timeToGetCoffee)
 {
     outtaHere = true;
 }
-       
+    
 if (timeToGetCoffee) {
 }
 else
-       
+    
 // comment on else case
-       
+    
 if (timeToGoHome)
     outtaHere = true;
-       </pre>
+    </pre>
 </li>
 </pre>
 </li>
@@ -123,36 +123,36 @@ void noArgFunction ();
 <ol>
 <li>In C++, the null pointer value should be written as <code>0</code>. In C it should be written as <code>NULL</code>. In Objective-C, it should be written as <code>nil</code> if it is being used as a null pointer of type <code>id</code> or another ObjC object type, otherwise <code>NULL</code>.</li>
 <li>True and false values of type <code>bool</code> (common in C and C++), or just generic true/false values, should be written as <code>true</code> and <code>false</code>. Values of the Objective-C <code>BOOL</code> type should be written as <code>YES</code> and <code>NO</code>.</li>
-<li>Tests for null pointers, false values and 0 values should all be done diretly, not through an inqueality or equality comparison.<br />
+<li>Tests for null pointers, false values and 0 values should all be done directly, not through an inqueality or equality comparison.<br />
 
 <h4 style="color: #008000 !important">Right:</h4>
 <pre style="background-color: #F2F2F2">
-       
+    
 // test for true
 if (foo->isSomething()) {
     // code
 }
-       
+    
 // test for false
 if (!foo->isSomething()) {
     // code
 }
-       
+    
 // test for non-null
 if (ptr) {
    // code
 }
-       
+    
 // test for null
 if (!ptr) {
    // code
 }
-       
+    
 // test for nonzero
 if (count) {
     // code
 }
-       
+    
 // test for zero
 if (!count) {
     // code
@@ -164,19 +164,19 @@ if (!count) {
 if (foo->isSomething() == true) {
     // code
 }
-       
+    
 if (foo->isSomething() != false) {
     // code
 }
-       
+    
 if (p == NULL) {
     // code
 }
-       
+    
 if (nil != p) {
     // code
 }
-       
+    
 if (count == 0) {
     // code
 }
@@ -197,7 +197,7 @@ class MyImportantClass;
 <h4 style="color: #ff0000 !important">Wrong:</h4>
 <pre style="background-color: #F2F2F2">
 class My_important_class;
-       
+    
 class myImportantClass;
 </pre>
 </li>
@@ -212,7 +212,7 @@ int myInt;
 <h4 style="color: #ff0000 !important">Wrong:</h4>
 <pre style="background-color: #F2F2F2">
 int MyInt;
-       
+    
 int my_int;
 </pre>
 </li>
@@ -291,5 +291,5 @@ Don't bother to organize them in a logical order.
 </ol>
 
 <?php
-       include("../footer.inc");
+    include("../footer.inc");
 ?>