Changeset 110582 in webkit

Timestamp:

Mar 13, 2012 11:10:40 AM (12 years ago)

Author:

commit-queue@webkit.org

Message:

Remove SlotVisitor::copy() method.
​https://bugs.webkit.org/show_bug.cgi?id=80973

Patch by Kwonjin Jeong <​gram@company100.net> on 2012-03-13
Reviewed by Geoffrey Garen.

SlotVisitor::copy() method isn't called anywhere.

• heap/MarkStack.cpp: Remove definition of SlotVisitor::copy() method.
• heap/SlotVisitor.h: Remove declaration of SlotVisitor::copy() method.

Location:

trunk/Source/JavaScriptCore

Files:

• ChangeLog (modified) (1 diff)
• heap/MarkStack.cpp (modified) (1 diff)
• heap/SlotVisitor.h (modified) (1 diff)

trunk/Source/JavaScriptCore/ChangeLog

@@ +1 @@
+2012-03-13  Kwonjin Jeong  <gram@company100.net>
+
+        Remove SlotVisitor::copy() method.
+        https://bugs.webkit.org/show_bug.cgi?id=80973
+
+        Reviewed by Geoffrey Garen.
+
+        SlotVisitor::copy() method isn't called anywhere.
+
+        * heap/MarkStack.cpp: Remove definition of SlotVisitor::copy() method.
+        * heap/SlotVisitor.h: Remove declaration of SlotVisitor::copy() method.
+
 2012-03-12  Hojong Han  <hojong.han@samsung.com>
 

trunk/Source/JavaScriptCore/heap/MarkStack.cpp

@@ -487 +487 @@
 }
 
-void SlotVisitor::copy(void** ptr, size_t bytes)
-{
-    void* newPtr = 0;
-    if (!(newPtr = allocateNewSpace(*ptr, bytes)))
-        return;
-
-    memcpy(newPtr, *ptr, bytes);
-    *ptr = newPtr;
-}
-
 void SlotVisitor::copyAndAppend(void** ptr, size_t bytes, JSValue* values, unsigned length)
 {

trunk/Source/JavaScriptCore/heap/SlotVisitor.h

@@ -63 +63 @@
 
     void startCopying();
-    void copy(void**, size_t);
     void copyAndAppend(void**, size_t, JSValue*, unsigned);
     void doneCopying();