Fix MSVC warnings about missing operator delete.
authorsanjoy@chromium.org <sanjoy@chromium.org@ce2b1a6d-e550-0410-aec6-3dcde31c8c00>
Mon, 18 Jun 2012 12:48:21 +0000 (12:48 +0000)
committersanjoy@chromium.org <sanjoy@chromium.org@ce2b1a6d-e550-0410-aec6-3dcde31c8c00>
Mon, 18 Jun 2012 12:48:21 +0000 (12:48 +0000)
BUG=
TEST=

Review URL: https://chromiumcodereview.appspot.com/10556038

git-svn-id: http://v8.googlecode.com/svn/branches/bleeding_edge@11847 ce2b1a6d-e550-0410-aec6-3dcde31c8c00

src/list.h
src/splay-tree.h

index 3ca4a3fba8a8d823016298239d72fdc233c10b92..7fd4f5cd2d6f6af7b8e2675252347b560c95dbdd 100644 (file)
@@ -74,6 +74,11 @@ class List {
     AllocationPolicy::Delete(p);
   }
 
+  // Please the MSVC compiler.  We should never have to execute this.
+  INLINE(void operator delete(void* p, AllocationPolicy allocator)) {
+    UNREACHABLE();
+  }
+
   // Returns a reference to the element at index i.  This reference is
   // not safe to use after operations that can change the list's
   // backing store (e.g. Add).
index 388f9b5429f889bc7aebcefaf90ba543a7f3178d..8844d8a8ffeb2d62a856ff19a1bdc2ff58e8aa24 100644 (file)
@@ -66,9 +66,13 @@ class SplayTree {
                             AllocationPolicy allocator = AllocationPolicy())) {
     return allocator.New(static_cast<int>(size));
   }
-  INLINE(void operator delete(void* p, size_t)) {
+  INLINE(void operator delete(void* p)) {
     AllocationPolicy::Delete(p);
   }
+  // Please the MSVC compiler.  We should never have to execute this.
+  INLINE(void operator delete(void* p, AllocationPolicy policy)) {
+    UNREACHABLE();
+  }
 
   // Inserts the given key in this tree with the given value.  Returns
   // true if a node was inserted, otherwise false.  If found the locator
@@ -119,9 +123,14 @@ class SplayTree {
     INLINE(void* operator new(size_t size, AllocationPolicy allocator)) {
       return allocator.New(static_cast<int>(size));
     }
-    INLINE(void operator delete(void* p, size_t)) {
+    INLINE(void operator delete(void* p)) {
       return AllocationPolicy::Delete(p);
     }
+    // Please the MSVC compiler.  We should never have to execute
+    // this.
+    INLINE(void operator delete(void* p, AllocationPolicy allocator)) {
+      UNREACHABLE();
+    }
 
     Key key() { return key_; }
     Value value() { return value_; }