Add coding style guidelines for 'using namespace.'
authorbeidson@apple.com <beidson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 22 Sep 2009 02:15:29 +0000 (02:15 +0000)
committerbeidson@apple.com <beidson@apple.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Tue, 22 Sep 2009 02:15:29 +0000 (02:15 +0000)
https://bugs.webkit.org/show_bug.cgi?id=29499

Reviewed by Darin Adler.

* coding/coding-style.html:

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

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

index 0b3736e..3e6e094 100644 (file)
@@ -1,5 +1,14 @@
 2009-09-21  Brady Eidson  <beidson@apple.com>
 
+        Reviewed by Darin Adler.
+
+        Add coding style guidelines for 'using namespace.'
+        https://bugs.webkit.org/show_bug.cgi?id=29499
+
+        * coding/coding-style.html:
+
+2009-09-21  Brady Eidson  <beidson@apple.com>
+
         Rubberstamped by Sam Weinig.
 
         Add some example files for an upcoming blog post.
index b2d4d94..0ad12a5 100644 (file)
@@ -704,7 +704,60 @@ Don't bother to organize them in a logical order.
 #include "QualifiedName.h"
 #include "Attribute.h"
 </pre>
+</ol>
+
+<h3>"using namespace" Statements</h3>
+
+<ol>
+
+<li>Any "using namespace" statements for a nested namespace whose parent namespace 
+is also defined in a file must be declared within that namespace definition.
+
+<h4 class="right">Right:</h4>
+<pre class="code">
+// HTMLBaseElement.cpp
+
+namespace WebCore {
+
+using namespace HTMLNames;
+
+} // namespace WebCore
+</pre>
+
+<h4 class="wrong">Wrong:</h4>
+<pre class="code">
+// HTMLBaseElement.cpp
+
+using namespace WebCore::HTMLNames;
+
+namespace WebCore {
 
+} // namespace WebCore
+</pre>
+
+<li>Any other "using namespace" statements must be declared before the first namespace 
+definition in the file.
+<h4 class="right">Right:</h4>
+<pre class="code">
+// HTMLSelectElement.cpp
+
+using namespace std;
+
+namespace WebCore {
+
+} // namespace WebCore
+</pre>
+
+<h4 class="wrong">Wrong:</h4>
+<pre class="code">
+// HTMLSelectElement.cpp
+
+namespace WebCore {
+
+using namespace std;
+
+} // namespace WebCore
+</pre>
 </ol>
 
 <?php