Revision: 10028
Author:   [email protected]
Date:     Fri Nov 18 03:08:46 2011
Log: Avoid explicitly using built-in new operator where possible to appease Windows build.

Review URL: http://codereview.chromium.org/8561011
http://code.google.com/p/v8/source/detail?r=10028

Modified:
 /branches/bleeding_edge/src/elements.cc
 /branches/bleeding_edge/src/extensions/gc-extension.cc
 /branches/bleeding_edge/src/hashmap.cc
 /branches/bleeding_edge/src/scopes.cc

=======================================
--- /branches/bleeding_edge/src/elements.cc     Fri Nov 18 00:59:33 2011
+++ /branches/bleeding_edge/src/elements.cc     Fri Nov 18 03:08:46 2011
@@ -827,7 +827,7 @@
     ELEMENTS_LIST(ACCESSOR_STRUCT)
 #undef ACCESSOR_STRUCT
   } element_accessors = {
-#define ACCESSOR_INIT(Class, Name) ::new Class(),
+#define ACCESSOR_INIT(Class, Name) new Class(),
     ELEMENTS_LIST(ACCESSOR_INIT)
 #undef ACCESSOR_INIT
   };
=======================================
--- /branches/bleeding_edge/src/extensions/gc-extension.cc Fri Nov 18 00:59:33 2011 +++ /branches/bleeding_edge/src/extensions/gc-extension.cc Fri Nov 18 03:08:46 2011
@@ -46,7 +46,8 @@


 void GCExtension::Register() {
-  static GCExtension* gc_extension = ::new GCExtension();
+  static GCExtension* gc_extension = NULL;
+  if (gc_extension == NULL) gc_extension = new GCExtension();
   static v8::DeclareExtension gc_extension_declaration(gc_extension);
 }

=======================================
--- /branches/bleeding_edge/src/hashmap.cc      Fri Nov 18 00:59:33 2011
+++ /branches/bleeding_edge/src/hashmap.cc      Fri Nov 18 03:08:46 2011
@@ -36,7 +36,7 @@
 namespace v8 {
 namespace internal {

-Allocator* HashMap::DefaultAllocator = ::new Allocator();
+Allocator* HashMap::DefaultAllocator = new Allocator();


 HashMap::HashMap(MatchFun match,
=======================================
--- /branches/bleeding_edge/src/scopes.cc       Fri Nov 18 00:59:33 2011
+++ /branches/bleeding_edge/src/scopes.cc       Fri Nov 18 03:08:46 2011
@@ -55,7 +55,7 @@
 };


-static ZoneAllocator* LocalsMapAllocator = ::new ZoneAllocator();
+static ZoneAllocator* LocalsMapAllocator = new ZoneAllocator();


// ----------------------------------------------------------------------------

--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to