Slight cleanup from code review feedback

Author: weswigham

src/compiler/emitter.ts

@@ -43,7 +43,7 @@ namespace ts {
     }
 
     /*@internal*/
-    export function getOutputPathsFor(sourceFile: SourceFile | Bundle, host: EmitHost, forceDtsPaths?: boolean) {
+    export function getOutputPathsFor(sourceFile: SourceFile | Bundle, host: EmitHost, forceDtsPaths: boolean) {
         const options = host.getCompilerOptions();
         if (sourceFile.kind === SyntaxKind.Bundle) {
             const jsFilePath = options.outFile || options.out;
@@ -129,7 +129,8 @@ namespace ts {
         let declarationPrinter: Printer;
         if (emitOnlyDtsFiles || compilerOptions.declaration) {
             const nonJsFiles = filter(sourceFiles, isSourceFileNotJavaScript);
-            declarationTransform = transformNodes(resolver, host, compilerOptions, (compilerOptions.outFile || compilerOptions.out) ? [createBundle(nonJsFiles)] : nonJsFiles, [transformDeclarations], /*allowDtsFiles*/ false);
+            const inputListOrBundle = (compilerOptions.outFile || compilerOptions.out) ? [createBundle(nonJsFiles)] : nonJsFiles;
+            declarationTransform = transformNodes(resolver, host, compilerOptions, inputListOrBundle, [transformDeclarations], /*allowDtsFiles*/ false);
             declarationPrinter = createPrinter({ ...compilerOptions, onlyPrintJsDocStyle: true } as PrinterOptions, {
                 // resolver hooks
                 hasGlobalName: resolver.hasGlobalName,
@@ -182,9 +183,9 @@ namespace ts {
                         return getOriginalNode(n) === getOriginalNode(sourceFileOrBundle);
                     });
                     if (associatedDeclarationTree) {
-                        const previousState = sourceMap.setDisabled(true);
+                        const previousState = sourceMap.setState(/*disabled*/ true);
                         printSourceFileOrBundle(declarationFilePath, /*sourceMapFilePath*/ undefined, associatedDeclarationTree, declarationPrinter, /*shouldSkipSourcemap*/ true);
-                        sourceMap.setDisabled(previousState);
+                        sourceMap.setState(previousState);
                     }
                     else {
                         Debug.fail(`No declaration output found for path "${declarationFilePath}" for js output file: "${jsFilePath}".`);

src/compiler/sourcemap.ts

@@ -71,7 +71,7 @@ namespace ts {
         /**
          * @returns the previous disabled state
          */
-        setDisabled(state: boolean): boolean;
+        setState(disabled: boolean): boolean;
     }
 
     // Used for initialize lastEncodedSourceMapSpan and reset lastEncodedSourceMapSpan when updateLastEncodedAndRecordedSpans
@@ -112,10 +112,10 @@ namespace ts {
             emitTokenWithSourceMap,
             getText,
             getSourceMappingURL,
-            setDisabled,
+            setState,
         };
 
-        function setDisabled(state: boolean) {
+        function setState(state: boolean) {
             const last = disabled;
             disabled = state;
             return last;

src/compiler/transformers/declarations.ts

@@ -184,7 +184,7 @@ namespace ts {
             possibleImports = undefined;
             importDeclarationMap = createMap();
             necessaryTypeRefernces = undefined;
-            const refs = collectReferences(currentSourceFile);
+            const refs = collectReferences(currentSourceFile, createMap());
             const references: FileReference[] = [];
             const outputFilePath = getDirectoryPath(normalizeSlashes(getOutputPathsFor(node, host, /*forceDtsPaths*/ true).declarationFilePath));
             const referenceVisitor = mapReferencesIntoArray(references, outputFilePath);
@@ -234,7 +234,7 @@ namespace ts {
             }
         }
 
-        function collectReferences(sourceFile: SourceFile, ret = createMap<SourceFile>()) {
+        function collectReferences(sourceFile: SourceFile, ret: Map<SourceFile>) {
             if (noResolve || isSourceFileJavaScript(sourceFile)) return ret;
             forEach(sourceFile.referencedFiles, f => {
                 const elem = tryResolveScriptReference(host, sourceFile, f);
@@ -497,7 +497,7 @@ namespace ts {
             }
             if (decl.importClause.namedBindings.kind === SyntaxKind.NamespaceImport) {
                 // Namespace import (optionally with visible default)
-                const namedBindings = resolver.isDeclarationVisible(decl.importClause.namedBindings as NamespaceImport) ? decl.importClause.namedBindings : /*namedBindings*/ undefined;
+                const namedBindings = resolver.isDeclarationVisible(decl.importClause.namedBindings) ? decl.importClause.namedBindings : /*namedBindings*/ undefined;
                 return visibleDefaultBinding || namedBindings ? updateImportDeclaration(decl, /*decorators*/ undefined, decl.modifiers, updateImportClause(
                         decl.importClause,
                         visibleDefaultBinding,
@@ -797,7 +797,7 @@ namespace ts {
 
             switch (input.kind) {
                 case SyntaxKind.ExportDeclaration: {
-                    if (!resultHasExternalModuleIndicator && isSourceFile(input.parent)) {
+                    if (isSourceFile(input.parent)) {
                         resultHasExternalModuleIndicator = true;
                     }
                     // Always visible if the parent node isn't dropped for being not visible
@@ -806,7 +806,7 @@ namespace ts {
                 }
                 case SyntaxKind.ExportAssignment: {
                     // Always visible if the parent node isn't dropped for being not visible
-                    if (!resultHasExternalModuleIndicator && isSourceFile(input.parent)) {
+                    if (isSourceFile(input.parent)) {
                         resultHasExternalModuleIndicator = true;
                     }
                     if (input.expression.kind === SyntaxKind.Identifier) {