Fixed linter warnings.

This commit is contained in:
Ron Buckton 2016-02-23 17:06:54 -08:00
parent 0d8e152d9d
commit 72bfd2f5b6
3 changed files with 20 additions and 28 deletions

View File

@ -338,7 +338,7 @@ namespace ts {
function visitExportDeclaration(node: ExportDeclaration): OneOrMany<Statement> {
if (contains(externalImports, node)) {
let generatedName = getGeneratedNameForNode(node);
const generatedName = getGeneratedNameForNode(node);
if (node.exportClause) {
const statements: Statement[] = [];
// export { x, y } from "mod";

View File

@ -37,7 +37,7 @@ namespace ts {
let contextObjectForFile: Identifier;
let exportedLocalNames: Identifier[];
let exportedFunctionDeclarations: ExpressionStatement[];
let noSubstitution: Map<boolean> = {};
const noSubstitution: Map<boolean> = {};
return transformSourceFile;
@ -992,7 +992,7 @@ namespace ts {
function hasExportedReferenceInArrayDestructuringElement(node: Expression): boolean {
if (isSpreadElementExpression(node)) {
let expression = node.expression;
const expression = node.expression;
return isIdentifier(expression) && isExportedBinding(expression);
}
else {
@ -1002,7 +1002,7 @@ namespace ts {
function hasExportedReferenceInDestructuringElement(node: Expression): boolean {
if (isBinaryExpression(node)) {
let left = node.left;
const left = node.left;
return node.operatorToken.kind === SyntaxKind.EqualsToken
&& isDestructuringPattern(left)
&& hasExportedReferenceInDestructuringPattern(left);
@ -1011,7 +1011,7 @@ namespace ts {
return isExportedBinding(node);
}
else if (isSpreadElementExpression(node)) {
let expression = node.expression;
const expression = node.expression;
return isIdentifier(expression) && isExportedBinding(expression);
}
else if (isDestructuringPattern(node)) {
@ -1052,7 +1052,7 @@ namespace ts {
}
function getExternalModuleNameLiteral(importNode: ImportDeclaration | ExportDeclaration | ImportEqualsDeclaration) {
let moduleName = getExternalModuleName(importNode);
const moduleName = getExternalModuleName(importNode);
if (moduleName.kind === SyntaxKind.StringLiteral) {
return tryRenameExternalModule(<StringLiteral>moduleName)
|| getSynthesizedNode(<StringLiteral>moduleName);
@ -1074,12 +1074,12 @@ namespace ts {
}
function getLocalNameTextForExternalImport(node: ImportDeclaration | ExportDeclaration | ImportEqualsDeclaration): string {
let name = getLocalNameForExternalImport(node);
const name = getLocalNameForExternalImport(node);
return name ? name.text : undefined;
}
function getLocalNameForExternalImport(node: ImportDeclaration | ExportDeclaration | ImportEqualsDeclaration): Identifier {
let namespaceDeclaration = getNamespaceDeclarationNode(node);
const namespaceDeclaration = getNamespaceDeclarationNode(node);
if (namespaceDeclaration && !isDefaultImport(node)) {
return createIdentifier(getSourceTextOfNodeFromSourceFile(currentSourceFile, namespaceDeclaration.name));
}
@ -1182,8 +1182,8 @@ namespace ts {
* @param node The declaration to export.
*/
function createDeclarationExport(node: DeclarationStatement) {
let declarationName = getDeclarationName(node);
let exportName = node.flags & NodeFlags.Default ? createLiteral("default") : declarationName;
const declarationName = getDeclarationName(node);
const exportName = node.flags & NodeFlags.Default ? createLiteral("default") : declarationName;
return createExportStatement(exportName, declarationName);
}
@ -1209,15 +1209,15 @@ namespace ts {
}
function collectDependencyGroups(externalImports: (ImportDeclaration | ImportEqualsDeclaration | ExportDeclaration)[]) {
let groupIndices: Map<number> = {};
let dependencyGroups: DependencyGroup[] = [];
for (let i = 0; i < externalImports.length; ++i) {
let externalImport = externalImports[i];
let externalModuleName = getExternalModuleNameLiteral(externalImport);
let text = externalModuleName.text;
const groupIndices: Map<number> = {};
const dependencyGroups: DependencyGroup[] = [];
for (let i = 0; i < externalImports.length; i++) {
const externalImport = externalImports[i];
const externalModuleName = getExternalModuleNameLiteral(externalImport);
const text = externalModuleName.text;
if (hasProperty(groupIndices, text)) {
// deduplicate/group entries in dependency list by the dependency name
let groupIndex = groupIndices[text];
const groupIndex = groupIndices[text];
dependencyGroups[groupIndex].externalImports.push(externalImport);
continue;
}
@ -1245,14 +1245,6 @@ namespace ts {
exportedLocalNames.push(name);
}
function hoistExportedVariableDeclaration(name: Identifier) {
}
function hoistExportedFunctionDeclaration(node: FunctionDeclaration) {
}
function recordExportedFunctionDeclaration(node: FunctionDeclaration) {
if (!exportedFunctionDeclarations) {
exportedFunctionDeclarations = [];
@ -1262,7 +1254,7 @@ namespace ts {
}
function hoistBindingElement(node: VariableDeclaration | BindingElement, isExported: boolean) {
let name = node.name;
const name = node.name;
if (isIdentifier(name)) {
hoistVariableDeclaration(getSynthesizedNode(name));
if (isExported) {

View File

@ -2869,7 +2869,7 @@ namespace ts {
const exportSpecifiers: Map<ExportSpecifier[]> = {};
let exportEquals: ExportAssignment = undefined;
let hasExportStars = false;
for (let node of sourceFile.statements) {
for (const node of sourceFile.statements) {
switch (node.kind) {
case SyntaxKind.ImportDeclaration:
if (!(<ImportDeclaration>node).importClause ||
@ -2928,7 +2928,7 @@ namespace ts {
}
export function copyPrologueDirectives(from: Statement[], to: Statement[]): number {
for (let i = 0; i < from.length; ++i) {
for (let i = 0; i < from.length; i++) {
if (isPrologueDirective(from[i])) {
addNode(to, from[i]);
}