http://codereview.chromium.org/6451003/diff/1/src/runtime.cc File src/runtime.cc (right):
http://codereview.chromium.org/6451003/diff/1/src/runtime.cc#newcode1093 src/runtime.cc:1093: if (result.is_null()) return Failure::Exception(); And yet, if we had done it here, we would have caught this much faster. I'll add them. http://codereview.chromium.org/6451003/diff/1/test/mjsunit/regress/regress-1105.js File test/mjsunit/regress/regress-1105.js (right): http://codereview.chromium.org/6451003/diff/1/test/mjsunit/regress/regress-1105.js#newcode31 test/mjsunit/regress/regress-1105.js:31: this.eval('__defineSetter__("x",function(){throw 42;});') Ah, that works too. This was a blind copy of the original example, using eval instead of load. Thanks. http://codereview.chromium.org/6451003/ -- v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev
