LGTM
http://codereview.chromium.org/7349/diff/1/2 File src/runtime.cc (right): http://codereview.chromium.org/7349/diff/1/2#newcode1263 Line 1263: if (pat.length() < 0) { I don't understand this. You assert above that pat.length() > 1, so how can you get to a situation where pat.length() < 0? I guess you mean to put in some other constant here (please name the constant when you do). http://codereview.chromium.org/7349 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
