Skip to content

Commit

Permalink
Merge pull request mozilla#418 from marco-c/dont_catch_in_vm_execute
Browse files Browse the repository at this point in the history
Don't catch exceptions in invoke*
  • Loading branch information
mykmelez committed Oct 9, 2014
2 parents c7179bb + e55e80d commit 254454a
Showing 1 changed file with 3 additions and 8 deletions.
11 changes: 3 additions & 8 deletions vm.js
Original file line number Diff line number Diff line change
Expand Up @@ -1061,14 +1061,9 @@ VM.execute = function(ctx) {

var alternateImpl = methodInfo.alternateImpl;
if (alternateImpl) {
try {
Instrument.callPauseHooks(ctx.current());
Instrument.measure(alternateImpl, ctx, methodInfo);
Instrument.callResumeHooks(ctx.current());
} catch (e) {
Instrument.callResumeHooks(ctx.current());
throw e;
}
Instrument.callPauseHooks(ctx.current());
Instrument.measure(alternateImpl, ctx, methodInfo);
Instrument.callResumeHooks(ctx.current());
break;
}
pushFrame(methodInfo, consumes);
Expand Down

0 comments on commit 254454a

Please sign in to comment.