Update code style guidelines for using namespace inside a namespace definition
authorHironori.Fujii@sony.com <Hironori.Fujii@sony.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 6 Sep 2018 02:54:58 +0000 (02:54 +0000)
committerHironori.Fujii@sony.com <Hironori.Fujii@sony.com@268f45cc-cd09-0410-ab3c-d52691b4dbfc>
Thu, 6 Sep 2018 02:54:58 +0000 (02:54 +0000)
https://bugs.webkit.org/show_bug.cgi?id=189298

Reviewed by Ryosuke Niwa.

"using namespace" statements should be inside namespace definitions for
unified source builds.

* code-style.md(#using-position):

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

Websites/webkit.org/ChangeLog
Websites/webkit.org/code-style.md

index f9cae2a..22f5e32 100644 (file)
@@ -1,3 +1,15 @@
+2018-09-05  Fujii Hironori  <Hironori.Fujii@sony.com>
+
+        Update code style guidelines for using namespace inside a namespace definition
+        https://bugs.webkit.org/show_bug.cgi?id=189298
+
+        Reviewed by Ryosuke Niwa.
+
+        "using namespace" statements should be inside namespace definitions for
+        unified source builds.
+
+        * code-style.md(#using-position): 
+
 2018-08-31  Dean Jackson  <dino@apple.com>
 
         Add support for USDZ to webkit.org for a sample file
index 7594067..0b71e8d 100644 (file)
@@ -1126,17 +1126,17 @@ namespace WebCore {
 } // namespace WebCore
 ```
 
-[](#using-position) In implementation files, put all other "using" statements at the beginning of the file, before any namespace definitions and after any "include" statements.
+[](#using-position) In implementation files, put all "using namespace" statements inside namespace definitions.
 
 ###### Right:
 
 ```cpp
 // HTMLSelectElement.cpp
 
-using namespace other;
-
 namespace WebCore {
 
+using namespace other;
+
 } // namespace WebCore
 ```
 
@@ -1145,10 +1145,10 @@ namespace WebCore {
 ```cpp
 // HTMLSelectElement.cpp
 
-namespace WebCore {
-
 using namespace other;
 
+namespace WebCore {
+
 } // namespace WebCore
 ```