Chromium Code Reviews

Issue 6259010: Change recursive error printing to just replace recursively (Closed)

Created:
9 years, 11 months ago by Mads Ager (chromium)
Modified:
9 years, 7 months ago
Reviewers:
Søren Thygesen Gjesse
CC:
v8-dev
Visibility:
Public.

Description

Change recursive error printing to just replace recursively encountered error objects with the empty string. This actually does match the Safari behaviour. Committed: http://code.google.com/p/v8/source/detail?r=6430

Patch Set 1 #

Unified diffs Side-by-side diffs Stats (+6 lines, -9 lines)
M src/messages.js View 1 chunk +4 lines, -7 lines 0 comments
M test/mjsunit/cyclic-error-to-string.js View 1 chunk +2 lines, -2 lines 0 comments

Messages

Total messages: 2 (0 generated)
Mads Ager (chromium)
9 years, 11 months ago (2011-01-21 14:00:27 UTC) #1
Søren Thygesen Gjesse
9 years, 11 months ago (2011-01-21 14:06:52 UTC) #2
LGTM

Powered by Google App Engine