Changeset 54258 in webkit

Timestamp:

Feb 2, 2010 3:16:35 PM (14 years ago)

Author:

snej@chromium.org

Message:

Clarified coding guidelines on wiki to allow indentation of _nested_ namespaces.
Reviewed by David Levin.
​https://bugs.webkit.org/show_bug.cgi?id=32137

Location:

trunk/WebKitSite

Files:

• ChangeLog (modified) (1 diff)
• coding/coding-style.html (modified) (3 diffs)

trunk/WebKitSite/ChangeLog

@@ +1 @@
+2010-02-02  Jens Alfke  <snej@chromium.org>
+
+        Reviewed by David Levin.
+
+        Clarified coding guidelines on wiki to allow indentation of
+        _nested_ namespaces.
+        https://bugs.webkit.org/show_bug.cgi?id=32137
+
+        * coding/coding-style.html:
+
 2010-01-24  Chris Jerdonek  <cjerdonek@webkit.org>
 

trunk/WebKitSite/coding/coding-style.html

@@ -46 +46 @@
 </pre>
 </li>
-<li>Code inside a namespace should not be indented.
+<li>The contents of an outermost <code>namespace</code> block (and any nested namespaces with the same scope)
+should not be indented. The contents of other nested namespaces should be indented.
 <h4 class="right">Right:</h4>
 <pre class="code">
@@ -56 +57 @@
     ...
 };
+
+namespace NestedNamespace {
+    ...
+}
 
 } // namespace WebCore
@@ -79 +84 @@
         ...
     };
+
+    namespace NestedNamespace {
+    ...
+    }
 
 } // namespace WebCore