Bug 74335

Summary: Remove most uses of JSArray vptr
Product: WebKit Reporter: Mark Hahnenberg <mhahnenberg>
Component: JavaScriptCoreAssignee: Mark Hahnenberg <mhahnenberg>
Status: RESOLVED FIXED    
Severity: Normal    
Priority: P2    
Version: 528+ (Nightly build)   
Hardware: Unspecified   
OS: Unspecified   
Bug Depends on: 74332    
Bug Blocks: 74331    
Attachments:
Description Flags
Remove uses of JSArray vptr ggaren: review+

Mark Hahnenberg
Reported 2011-12-12 14:26:53 PST
Similar to bug 74333, but we can't remove all of the uses because certain functions require other vptrs to be removed as well.
Attachments
Remove uses of JSArray vptr (38.05 KB, patch)
2011-12-12 14:48 PST, Mark Hahnenberg 		ggaren: review+
DetailsFormatted DiffDiff
View All Add attachment proposed patch, testcase, etc.
Mark Hahnenberg
Comment 1 2011-12-12 14:48:54 PST
Created attachment 118858 [details] Remove uses of JSArray vptr No changelog (see bug 74331)
Geoffrey Garen
Comment 2 2011-12-12 14:58:57 PST
Comment on attachment 118858 [details] Remove uses of JSArray vptr r=me
Geoffrey Garen
Comment 3 2011-12-12 15:00:21 PST
Comment on attachment 118858 [details] Remove uses of JSArray vptr View in context: https://bugs.webkit.org/attachment.cgi?id=118858&action=review > Source/JavaScriptCore/heap/MarkStack.cpp:315 > + if (cell->classInfo() == &JSArray::s_info) { Should use isJSArray().
Note You need to log in before you can comment on or make changes to this bug.