Reviewers: Mads Ager, Description: Remove unused runtime function.
The CompileScript runtime function is not used by any builtins any more. Please review this at http://codereview.chromium.org/80003 SVN Base: http://v8.googlecode.com/svn/branches/bleeding_edge/ Affected files: M src/runtime.h M src/runtime.cc Index: src/runtime.cc =================================================================== --- src/runtime.cc (revision 1743) +++ src/runtime.cc (working copy) @@ -4893,26 +4893,6 @@ } -static Object* Runtime_CompileScript(Arguments args) { - HandleScope scope; - ASSERT(args.length() == 4); - - CONVERT_ARG_CHECKED(String, source, 0); - CONVERT_ARG_CHECKED(String, script, 1); - CONVERT_CHECKED(Smi, line_attrs, args[2]); - int line = line_attrs->value(); - CONVERT_CHECKED(Smi, col_attrs, args[3]); - int col = col_attrs->value(); - Handle<JSFunction> boilerplate = - Compiler::Compile(source, script, line, col, NULL, NULL); - if (boilerplate.is_null()) return Failure::Exception(); - Handle<JSFunction> fun = - Factory::NewFunctionFromBoilerplate(boilerplate, - Handle<Context>(Top::context())); - return *fun; -} - - static Object* Runtime_SetNewFunctionAttributes(Arguments args) { // This utility adjusts the property attributes for newly created Function // object ("new Function(...)") by changing the map. Index: src/runtime.h =================================================================== --- src/runtime.h (revision 1743) +++ src/runtime.h (working copy) @@ -195,7 +195,6 @@ \ /* Globals */ \ F(CompileString, 2) \ - F(CompileScript, 4) \ F(GlobalPrint, 1) \ \ /* Eval */ \ --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
