[SCM] WebKit Debian packaging branch, webkit-1.1, updated. upstream/1.1.19-706-ge5415e9

snej at chromium.org snej at chromium.org
Thu Feb 4 21:37:17 UTC 2010


The following commit has been merged in the webkit-1.1 branch:
commit decfe6ff9384ff3bc0f8d8dc22890f2e69f96b96
Author: snej at chromium.org <snej at chromium.org@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Date:   Tue Feb 2 23:16:35 2010 +0000

    Clarified coding guidelines on wiki to allow indentation of _nested_ namespaces.
    Reviewed by David Levin.
    https://bugs.webkit.org/show_bug.cgi?id=32137
    
    git-svn-id: http://svn.webkit.org/repository/webkit/trunk@54258 268f45cc-cd09-0410-ab3c-d52691b4dbfc

diff --git a/WebKitSite/ChangeLog b/WebKitSite/ChangeLog
index 044fb1d..9aea763 100644
--- a/WebKitSite/ChangeLog
+++ b/WebKitSite/ChangeLog
@@ -1,3 +1,13 @@
+2010-02-02  Jens Alfke  <snej at 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 at webkit.org>
 
         Reviewed by David Kilzer.
diff --git a/WebKitSite/coding/coding-style.html b/WebKitSite/coding/coding-style.html
index abd1b72..7bc3c7f 100644
--- a/WebKitSite/coding/coding-style.html
+++ b/WebKitSite/coding/coding-style.html
@@ -45,7 +45,8 @@ int main()
 }
 </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">
 // Document.h
@@ -56,6 +57,10 @@ class Document {
     ...
 };
 
+namespace NestedNamespace {
+    ...
+}
+
 } // namespace WebCore
 
 // Document.cpp
@@ -79,6 +84,10 @@ namespace WebCore {
         ...
     };
 
+    namespace NestedNamespace {
+    ...
+    }
+
 } // namespace WebCore
 
 // Document.cpp

-- 
WebKit Debian packaging



More information about the Pkg-webkit-commits mailing list