microsoft/TypeScript
1
0
Fork 0
mirror of https://github.com/microsoft/TypeScript.git synced 2026-06-16 07:13:43 -05:00
Code Issues Packages Projects Releases 212 Wiki Activity

Add boolean flag to not walk the tree if there is no dynamic import

Browse Source
This commit is contained in:
Kanchalai Tanglertsampan
2017-04-04 15:43:45 -07:00
parent faaa38d93e
commit 55430c46e6
4 changed files with 15 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

10
src/compiler/program.ts
View File

@@ -1223,7 +1223,9 @@ namespace ts {
for (const node of file.statements) {
collectModuleReferences(node, /*inAmbientModule*/ false);
collectImportOrRequireCalls(node);
if (file.possiblyContainDynamicImport || isJavaScriptFile) {
collectDynamicImportOrRequireCalls(node);
}
}
file.imports = imports || emptyArray;
@@ -1285,8 +1287,8 @@ namespace ts {
}
}
function collectImportOrRequireCalls(node: Node): void {
if (isJavaScriptFile && isRequireCall(node, /*checkArgumentIsStringLiteral*/true)) {
function collectDynamicImportOrRequireCalls(node: Node): void {
if (isRequireCall(node, /*checkArgumentIsStringLiteral*/true)) {
(imports || (imports = [])).push(<StringLiteral>(<CallExpression>node).arguments[0]);
}
// we have to check the argument list has length of 1. We will still have to process these even though we have parsing error.
@@ -1294,7 +1296,7 @@ namespace ts {
(imports || (imports = [])).push(<StringLiteral>(<CallExpression>node).arguments[0]);
}
else {
forEachChild(node, collectImportOrRequireCalls);
forEachChild(node, collectDynamicImportOrRequireCalls);
}
}
}