Reviewers: Mads Ager, Description: Remove obsolete function NewArrayLiteral.
This function (which creates a tenured JS array) was not use by any code. Please review this at http://codereview.chromium.org/28070 SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/ Affected files: M src/factory.h M src/factory.cc Index: src/factory.h =================================================================== --- src/factory.h (revision 1352) +++ src/factory.h (working copy) @@ -186,10 +186,6 @@ // runtime. static Handle<JSObject> NewJSObjectFromMap(Handle<Map> map); - // Allocate a JS array representing an array literal. The array is - // pretenured (allocated directly in the old generation). - static Handle<JSArray> NewArrayLiteral(int length); - // JS arrays are pretenured when allocated by the parser. static Handle<JSArray> NewJSArray(int init_length, PretenureFlag pretenure = NOT_TENURED); Index: src/factory.cc =================================================================== --- src/factory.cc (revision 1352) +++ src/factory.cc (working copy) @@ -601,11 +601,6 @@ } -Handle<JSArray> Factory::NewArrayLiteral(int length) { - return NewJSArrayWithElements(NewFixedArray(length), TENURED); -} - - Handle<JSArray> Factory::NewJSArray(int length, PretenureFlag pretenure) { Handle<JSObject> obj = NewJSObject(Top::array_function(), pretenure); --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
