LGTM Please update the v8_base_arm.vcproj Visual Studio project file as well.
http://codereview.chromium.org/62146/diff/4001/4008 File src/func-name-inferrer.cc (right): http://codereview.chromium.org/62146/diff/4001/4008#newcode39 Line 39: // and starts with a capital letter. Should we enforce the capital first letter? http://codereview.chromium.org/62146/diff/4001/4015 File src/rewriter.cc (left): http://codereview.chromium.org/62146/diff/4001/4015#oldcode798 Line 798: if (!scope->is_global_scope()) { Any idea why the global scope used to be skipped here? http://codereview.chromium.org/62146 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
