LGTM

http://codereview.chromium.org/7263/diff/203/5
File src/runtime.cc (right):

http://codereview.chromium.org/7263/diff/203/5#newcode2162
Line 2162: next_length = mapping->get(next, 0, chars);
Missing comment here to explain why 0 is OK.

http://codereview.chromium.org/7263

--~--~---------~--~----~------------~-------~--~----~
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
-~----------~----~----~----~------~----~------~--~---

Reply via email to