http://codereview.chromium.org/3411013/diff/1/2 File src/runtime.cc (right):
http://codereview.chromium.org/3411013/diff/1/2#newcode8948 src/runtime.cc:8948: // args[0]: boolean indicating uncaught exceptions Comment is wrong too: It's not a boolean. Reworded. http://codereview.chromium.org/3411013/diff/1/2#newcode8950 src/runtime.cc:8950: static Object* Runtime_ChangeBreakOnException(Arguments args) { Contrary to what I said offline, the exception in fuzz-natives.js can be removed (which means that the comment in fuzz-natives.js didn't apply to this function - it does no error checking in recursive calls). http://codereview.chromium.org/3411013/diff/1/2#newcode8955 src/runtime.cc:8955: The boolean is ok, the number conversion introduces an unnecessary intermediate variable. http://codereview.chromium.org/3411013/show -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
