Commit 8557597
vm: don't print out arrow message for custom error
In `AppendExceptionLine()`, which is used both by the `vm`
module and the uncaught exception handler, don’t print anything
to stderr when called from the `vm` module, even if the
thrown object is not a native error instance.
Fixes: #7397
PR-URL: #7398
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>1 parent 7dbb0d0 commit 8557597Copy full SHA for 8557597
File tree
Expand file treeCollapse file tree
4 files changed
+42
-14
lines changedOpen diff view settings
Filter options
- src
- test/message
Expand file treeCollapse file tree
4 files changed
+42
-14
lines changedOpen diff view settings
Collapse file
+18-13Lines changed: 18 additions & 13 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
1439 | 1439 | |
1440 | 1440 | |
1441 | 1441 | |
1442 | | - |
| 1442 | + |
| 1443 | + |
1443 | 1444 | |
1444 | 1445 | |
1445 | 1446 | |
| ||
1521 | 1522 | |
1522 | 1523 | |
1523 | 1524 | |
1524 | | - |
1525 | | - |
1526 | | - |
1527 | | - |
1528 | | - |
1529 | | - |
| 1525 | + |
| 1526 | + |
| 1527 | + |
| 1528 | + |
| 1529 | + |
| 1530 | + |
| 1531 | + |
| 1532 | + |
| 1533 | + |
| 1534 | + |
1530 | 1535 | |
1531 | | - |
1532 | | - |
| 1536 | + |
| 1537 | + |
1533 | 1538 | |
1534 | | - |
1535 | | - |
1536 | | - |
| 1539 | + |
| 1540 | + |
| 1541 | + |
1537 | 1542 | |
1538 | 1543 | |
1539 | 1544 | |
| ||
1542 | 1547 | |
1543 | 1548 | |
1544 | 1549 | |
1545 | | - |
| 1550 | + |
1546 | 1551 | |
1547 | 1552 | |
1548 | 1553 | |
|
Collapse file
+3-1Lines changed: 3 additions & 1 deletion
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
126 | 126 | |
127 | 127 | |
128 | 128 | |
| 129 | + |
129 | 130 | |
130 | 131 | |
131 | | - |
| 132 | + |
| 133 | + |
132 | 134 | |
133 | 135 | |
134 | 136 | |
|
Collapse file
test/message/vm_caught_custom_runtime_error.js
Copy file name to clipboard+18Lines changed: 18 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
| 1 | + |
| 2 | + |
| 3 | + |
| 4 | + |
| 5 | + |
| 6 | + |
| 7 | + |
| 8 | + |
| 9 | + |
| 10 | + |
| 11 | + |
| 12 | + |
| 13 | + |
| 14 | + |
| 15 | + |
| 16 | + |
| 17 | + |
| 18 | + |
Collapse file
test/message/vm_caught_custom_runtime_error.out
Copy file name to clipboard+3Lines changed: 3 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
| 1 | + |
| 2 | + |
| 3 | + |
0 commit comments