LGTM Maybe extend the ExceptionOrder test in test-api, as it should have caught this.
http://codereview.chromium.org/43070/diff/1/4 File src/top.h (right): http://codereview.chromium.org/43070/diff/1/4#newcode161 Line 161: if (!thread_local_.external_caught_exception_) { Is this if just to avoid recalculation, or is there a subtle reason for it? Good to get rid of IsTheHole in here. http://codereview.chromium.org/43070 --~--~---------~--~----~------------~-------~--~----~ v8-dev mailing list [email protected] http://groups.google.com/group/v8-dev -~----------~----~----~----~------~----~------~--~---
