Split nested namespaces declaration in two lines in accordance with C++ Style Guide.

src/accessors.h

 // Copyright 2006-2008 the V8 project authors. All rights reserved.
 // Redistribution and use in source and binary forms, with or without
 // modification, are permitted provided that the following conditions are
 // met:
 //
 //     * Redistributions of source code must retain the above copyright
 //       notice, this list of conditions and the following disclaimer.
 //     * Redistributions in binary form must reproduce the above
 //       copyright notice, this list of conditions and the following
 //       disclaimer in the documentation and/or other materials provided
@@ skipping 10 matching lines @@
 // SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
 // LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
 // DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
 // THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
 // (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
 // OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
 
 #ifndef V8_ACCESSORS_H_
 #define V8_ACCESSORS_H_
 
-namespace v8 { namespace internal {
+namespace v8 {
+namespace internal {
 
 // The list of accessor descriptors. This is a second-order macro
 // taking a macro to be applied to all accessor descriptor names.
 #define ACCESSOR_DESCRIPTOR_LIST(V) \
   V(FunctionPrototype)   \
   V(FunctionLength)      \
   V(FunctionName)        \
   V(FunctionArguments)   \
   V(FunctionCaller)      \
   V(ArrayLength)         \
@@ skipping 51 matching lines @@
   static Object* ObjectGetPrototype(Object* receiver, void*);
   static Object* ObjectSetPrototype(JSObject* receiver, Object* value, void*);
 
   // Helper functions.
   static Object* FlattenNumber(Object* value);
   static Object* IllegalSetter(JSObject*, Object*, void*);
   static Object* IllegalGetAccessor(Object* object, void*);
   static Object* ReadOnlySetAccessor(JSObject*, Object* value, void*);
 };
 
 } }  // namespace v8::internal

[Mads Ager (chromium), 2009/05/25 09:49:56]

Shouldn't we fix this as well or is it usual to have these on the same line?

}  // namespace internal
}  // namespace v8

[Mikhail Naganov, 2009/05/25 09:59:34]

Craig Silverstein said that there's no agreement on this part yet. I've checked
Google's codebase---both 2-line and 1-line styles are used, so our style is OK
(at least for now). Personally, I like it more that 2-line style.

 
 #endif  // V8_ACCESSORS_H_