LGTM.
http://codereview.chromium.org/522015/diff/1/2 File src/runtime.cc (right): http://codereview.chromium.org/522015/diff/1/2#newcode2778 src/runtime.cc:2778: } else if (args[0]->IsString() && Fits nicely on one line, right? http://codereview.chromium.org/522015/diff/1/2#newcode2780 src/runtime.cc:2780: // Fast case for string indexing using [] with a SMI index. SMI -> Smi or smi http://codereview.chromium.org/522015 -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
