Skip to content

repl: hoist declarations when processing top level await #39265

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
ejose19 wants to merge 11 commits into from
Closed

repl: hoist declarations when processing top level await #39265

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
a1d5b9c
repl: hoist declarations when processing top level await
ejose19 Jul 5, 2021
67cf682
refactor: improve variableIdentifiersToHoist registering
ejose19 Jul 5, 2021
79e59cf
refactor: categorize variable hoisting by var & let
ejose19 Jul 5, 2021
f5f305e
test: adjust & add more test cases
ejose19 Jul 5, 2021
a0ca702
repl: improve variable declaration hoisting under for of/in & for awa…
ejose19 Jul 5, 2021
12b4f06
repl: restore unconditionally hoisting of function declaration
ejose19 Jul 5, 2021
ecd82b5
test: add "for of/in" & "for await of" cases
ejose19 Jul 5, 2021
7159b9c
test: adjust function declaration cases
ejose19 Jul 5, 2021
710ad75
chore: apply eslint fixes
ejose19 Jul 5, 2021
617d735
repl: use "var" for function hoist
ejose19 Jul 5, 2021
9bb9c3d
repl: fix hoisted declaration statements order
ejose19 Jul 6, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
repl: use "var" for function hoist
  • Loading branch information
@ejose19
ejose19 committed Jul 5, 2021
commit 617d7352bd627a502689f8111e8b0d0175b04ff2
2 changes: 1 addition & 1 deletion lib/internal/repl/await.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,7 +45,7 @@ const visitorsWithoutAncestors = {
},
FunctionDeclaration(node, state, c) {
state.prepend(node, `${node.id.name}=`);
state.prepend(state.wrapper, `let ${node.id.name}; `);
state.prepend(state.wrapper, `var ${node.id.name}; `);
},
FunctionExpression: noop,
ArrowFunctionExpression: noop,
Expand Down
4 changes: 2 additions & 2 deletions test/parallel/test-repl-preprocess-top-level-await.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -54,11 +54,11 @@ const testCases = [
'(async () => { return (console.log(`${(await { a: 1 }).a}`)) })()' ],
/* eslint-enable no-template-curly-in-string */
[ 'await 0; function foo() {}',
'let foo; (async () => { await 0; foo=function foo() {} })()' ],
'var foo; (async () => { await 0; foo=function foo() {} })()' ],
[ 'await 0; class Foo {}',
'let Foo; (async () => { await 0; Foo=class Foo {} })()' ],
[ 'if (await true) { function foo() {} }',
'let foo; (async () => { if (await true) { foo=function foo() {} } })()' ],
'var foo; (async () => { if (await true) { foo=function foo() {} } })()' ],
[ 'if (await true) { class Foo{} }',
'(async () => { if (await true) { class Foo{} } })()' ],
[ 'if (await true) { var a = 1; }',
Expand Down
3 changes: 1 addition & 2 deletions test/parallel/test-repl-top-level-await.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -113,8 +113,7 @@ async function ordinaryTests() {
['for (let i = 0; i < 2; ++i) await i'],
['for (let i = 0; i < 2; ++i) { await i }'],
['await 0', '0'],
['await 0; function foo() {}',
"Uncaught SyntaxError: Identifier 'foo' has already been declared"],
['await 0; function foo() {}'],
['foo', '[Function: foo]'],
['class Foo {}; await 1;', '1'],
['Foo', '[class Foo]'],
Expand Down