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

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
refactor: categorize variable hoisting by var & let
  • Loading branch information
@ejose19
ejose19 committed Jul 5, 2021
commit 79e59cf5d8781f0a88e3b043bc91399bcd36f9af
109 changes: 61 additions & 48 deletions lib/internal/repl/await.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -22,11 +22,17 @@ const parser = require('internal/deps/acorn/acorn/dist/acorn').Parser;
const walk = require('internal/deps/acorn/acorn-walk/dist/walk');
const { Recoverable } = require('internal/repl');

function isTopLevelDeclaration(state) {
return state.ancestors[state.ancestors.length - 2] === state.body;
}

const noop = FunctionPrototype;
const visitorsWithoutAncestors = {
ClassDeclaration(node, state, c) {
state.prepend(node, `${node.id.name}=`);
state.prepend(state.root.body[0], `let ${node.id.name}; `);
if (isTopLevelDeclaration(state)) {
state.prepend(node, `${node.id.name}=`);
state.prepend(state.wrapper, `let ${node.id.name}; `);
}

walk.base.ClassDeclaration(node, state, c);
},
Expand All @@ -37,8 +43,10 @@ const visitorsWithoutAncestors = {
walk.base.ForOfStatement(node, state, c);
},
FunctionDeclaration(node, state, c) {
state.prepend(node, `${node.id.name}=`);
state.prepend(state.root.body[0], `let ${node.id.name}; `);
if (isTopLevelDeclaration(state)) {
state.prepend(node, `${node.id.name}=`);
state.prepend(state.wrapper, `let ${node.id.name}; `);
}
},
FunctionExpression: noop,
ArrowFunctionExpression: noop,
Expand All @@ -52,57 +60,62 @@ const visitorsWithoutAncestors = {
walk.base.ReturnStatement(node, state, c);
},
VariableDeclaration(node, state, c) {
if (node.declarations.length === 1) {
state.replace(node.start, node.start + node.kind.length, 'void');
} else {
state.replace(node.start, node.start + node.kind.length, 'void (');
}
const variableKind = node.kind;
if (variableKind === 'var' || isTopLevelDeclaration(state)) {
if (node.declarations.length === 1) {
state.replace(node.start, node.start + node.kind.length, 'void');
} else {
state.replace(node.start, node.start + node.kind.length, 'void (');
}

ArrayPrototypeForEach(node.declarations, (decl) => {
state.prepend(decl, '(');
state.append(decl, decl.init ? ')' : '=undefined)');
});
ArrayPrototypeForEach(node.declarations, (decl) => {
state.prepend(decl, '(');
state.append(decl, decl.init ? ')' : '=undefined)');
});

if (node.declarations.length !== 1) {
state.append(node.declarations[node.declarations.length - 1], ')');
}
if (node.declarations.length !== 1) {
state.append(node.declarations[node.declarations.length - 1], ')');
}

const variableIdentifiersToHoist = [];
function registerVariableDeclarationIdentifiers(node) {
switch (node.type) {
case 'Identifier':
ArrayPrototypePush(variableIdentifiersToHoist, node.name);
break;
case 'ObjectPattern':
ArrayPrototypeForEach(
node.properties,
(property) => {
const variableIdentifiersToHoist = [
['var', []],
['let', []],
];
function registerVariableDeclarationIdentifiers(node) {
switch (node.type) {
case 'Identifier':
ArrayPrototypePush(
variableIdentifiersToHoist[variableKind === 'var' ? 0 : 1][1],
node.name
);
break;
case 'ObjectPattern':
ArrayPrototypeForEach(node.properties, (property) => {
registerVariableDeclarationIdentifiers(property.value);
}
);
break;
case 'ArrayPattern':
ArrayPrototypeForEach(
node.elements,
(element) => {
});
break;
case 'ArrayPattern':
ArrayPrototypeForEach(node.elements, (element) => {
registerVariableDeclarationIdentifiers(element);
}
);
break;
});
break;
}
}
}

ArrayPrototypeForEach(
node.declarations,
(decl) => {
ArrayPrototypeForEach(node.declarations, (decl) => {
registerVariableDeclarationIdentifiers(decl.id);
}
);

if (variableIdentifiersToHoist.length > 0) {
state.prepend(
state.root.body[0],
'let ' + ArrayPrototypeJoin(variableIdentifiersToHoist, ', ') + '; '
});

ArrayPrototypeForEach(
variableIdentifiersToHoist,
({ 0: kind, 1: identifiers }) => {
if (identifiers.length > 0) {
state.prepend(
state.wrapper,
`${kind} ${ArrayPrototypeJoin(identifiers, ', ')}; `
);
}
}
);
}

Expand Down Expand Up @@ -163,7 +176,7 @@ function processTopLevelAwait(src) {
}
const body = root.body[0].expression.callee.body;
const state = {
root,
wrapper: root.body[0],
body,
ancestors: [],
replace(from, to, str) {
Expand Down