Commit 7e8a00a
readline: fix question stack overflow
This commit fixes readline interface's question callback wrapping when
it is being called with `signal` option. Previous version completely
overwrites passed callback and throws "Maximum call stack size exceeded"
error.
PR-URL: #43320
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>1 parent 112518f commit 7e8a00aCopy full SHA for 7e8a00a
File tree
Expand file treeCollapse file tree
3 files changed
+37
-1
lines changedOpen diff view settings
Filter options
- lib
- test/parallel
Expand file treeCollapse file tree
3 files changed
+37
-1
lines changedOpen diff view settings
Collapse file
+2-1Lines changed: 2 additions & 1 deletion
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
148 | 148 | |
149 | 149 | |
150 | 150 | |
| 151 | + |
151 | 152 | |
152 | 153 | |
153 | | - |
| 154 | + |
154 | 155 | |
155 | 156 | |
156 | 157 | |
|
Collapse file
test/parallel/test-readline-interface.js
Copy file name to clipboardExpand all lines: test/parallel/test-readline-interface.js+24Lines changed: 24 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
1006 | 1006 | |
1007 | 1007 | |
1008 | 1008 | |
| 1009 | + |
| 1010 | + |
| 1011 | + |
| 1012 | + |
| 1013 | + |
| 1014 | + |
| 1015 | + |
| 1016 | + |
| 1017 | + |
| 1018 | + |
| 1019 | + |
1009 | 1020 | |
1010 | 1021 | |
1011 | 1022 | |
| ||
1030 | 1041 | |
1031 | 1042 | |
1032 | 1043 | |
| 1044 | + |
| 1045 | + |
| 1046 | + |
| 1047 | + |
| 1048 | + |
| 1049 | + |
| 1050 | + |
| 1051 | + |
| 1052 | + |
| 1053 | + |
| 1054 | + |
| 1055 | + |
| 1056 | + |
1033 | 1057 | |
1034 | 1058 | |
1035 | 1059 | |
|
Collapse file
test/parallel/test-readline-promises-interface.js
Copy file name to clipboardExpand all lines: test/parallel/test-readline-promises-interface.js+11Lines changed: 11 additions & 0 deletions
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| ||
909 | 909 | |
910 | 910 | |
911 | 911 | |
| 912 | + |
| 913 | + |
| 914 | + |
| 915 | + |
| 916 | + |
| 917 | + |
| 918 | + |
| 919 | + |
| 920 | + |
| 921 | + |
| 922 | + |
912 | 923 | |
913 | 924 | |
914 | 925 | |
|
0 commit comments