2006-10-25 Mark Rowe <bdash@webkit.org>
authorbdash <bdash@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 25 Oct 2006 08:22:35 +0000 (08:22 +0000)
committerbdash <bdash@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Wed, 25 Oct 2006 08:22:35 +0000 (08:22 +0000)
        Reviewed by Maciej.

        Fix some HTML bugs in the coding style guidelines.

        * coding/coding-style.html:

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

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

index 9f5f32a53ba76cb7066fb8e9290c4c4eded31907..aa5e5f6973f776b65f2ddac4ccd82a83bfb2dcde 100644 (file)
@@ -1,3 +1,11 @@
+2006-10-25  Mark Rowe  <bdash@webkit.org>
+
+        Reviewed by Maciej.
+
+        Fix some HTML bugs in the coding style guidelines.
+
+        * coding/coding-style.html:
+
 2006-10-13  Geoffrey Garen  <ggaren@apple.com>
 
         Reviewed by Beth.
index cd851042bd0a9d2e6e485e04c6cb31cd750df657..15e91df28253c7bc1df7bdd84c8e3aea034a3d4a 100644 (file)
@@ -53,14 +53,14 @@ void foo() {
 
 <h4 class="right">Right:</h4>
 <pre class="code">
-for (int i = 0; i < 10; i++) {
+for (int i = 0; i &lt; 10; i++) {
     // do stuff
 }
 </pre>
 
 <h4 class="wrong">Wrong:</h4>
 <pre class="code">
-for (int i = 0; i < 10; i++)
+for (int i = 0; i &lt; 10; i++)
 {
     // do stuff
 }
@@ -242,16 +242,16 @@ they should read like a phrase or sentence and each piece of the selector should
 <h4 class="right">Right:</h4>
 <pre class="code">
 #define WBStopButtonTitle() \
-        NSLocalizedString(@\"Stop\", @\"Stop button title\")
+        NSLocalizedString(@"Stop", @"Stop button title")
 </pre>
 
 <h4 class="wrong">Wrong:</h4>
 <pre class="code">
 #define WB_STOP_BUTTON_TITLE \
-        NSLocalizedString(@\"Stop\", @\"Stop button\")
+        NSLocalizedString(@"Stop", @"Stop button title")
 
 #define WBStopButtontitle \
-        NSLocalizedString(@\"Stop\", @\"Stop button\")
+        NSLocalizedString(@"Stop", @"Stop button title")
 </pre>
 </li>
 
@@ -314,18 +314,18 @@ should be written with no space between the type name and the * or &amp;.
 
 <h4 class="right">Right:</h4>
 <pre class="code">
-Image* SVGStyledElement::doSomething(PaintInfo& paintInfo)
+Image* SVGStyledElement::doSomething(PaintInfo&amp; paintInfo)
 {
-  SVGStyledElement* element = static_cast<SVGStyledElement*>(node());
-  const KCDashArray& dashes = dashArray();
+  SVGStyledElement* element = static_cast&lt;SVGStyledElement*>(node());
+  const KCDashArray&amp; dashes = dashArray();
 </pre>
 
 <h4 class="wrong">Wrong:</h4>
 <pre class="code">
-Image *SVGStyledElement::doSomething(PaintInfo &paintInfo)
+Image *SVGStyledElement::doSomething(PaintInfo &amp;paintInfo)
 {
-    SVGStyledElement *element = static_cast<SVGStyledElement *>(node());
-    const KCDashArray &dashes = dashArray();
+    SVGStyledElement *element = static_cast&lt;SVGStyledElement *>(node());
+    const KCDashArray &amp;dashes = dashArray();
 </pre>
 
 <h3>Include Statements</h3>