We just return a failure if there is a stack overflow exception. Apparently
that is what the old code did too :-/

No, the old code did:
  if (result == EXCEPTION && !Top::has_pending_exception()) {
    // We detected a stack overflow (on the backtrack stack) in RegExp code,
    // but haven't created the exception yet.
    Top::StackOverflow();
  }
[from NativeRegExpMacroAssembler::Execute(...) in regexp-macro-assembler.cc]


http://codereview.chromium.org/521028
-- 
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev

Reply via email to