Code review

Author: andy-ms

src/compiler/core.ts

@@ -394,9 +394,9 @@ namespace ts {
         return result;
     }
 
-    export function mapIter<T, U>(iter: Iterator<T>, mapFn: (x: T) => U): Iterator<U> {
+    export function mapIterator<T, U>(iter: Iterator<T>, mapFn: (x: T) => U): Iterator<U> {
         return { next };
-        function  next(): { value: U, done: false } | { value: never, done: true } {
+        function next(): { value: U, done: false } | { value: never, done: true } {
             const iterRes = iter.next();
             return iterRes.done ? iterRes : { value: mapFn(iterRes.value), done: false };
         }
@@ -942,7 +942,7 @@ namespace ts {
         }
     }
 
-    export function arrayIter<T>(array: ReadonlyArray<T>): Iterator<T> {
+    export function arrayIterator<T>(array: ReadonlyArray<T>): Iterator<T> {
         let i = 0;
         return { next: () => {
             if (i === array.length) {

src/services/codefixes/importFixes.ts

@@ -316,7 +316,7 @@ namespace ts.codefix {
     }
 
     export function getModuleSpecifierForNewImport(sourceFile: SourceFile, moduleSymbols: ReadonlyArray<Symbol>, options: CompilerOptions, getCanonicalFileName: (file: string) => string, host: LanguageServiceHost): string | undefined {
-        const choices = mapIter(arrayIter(moduleSymbols), moduleSymbol => {
+        const choices = mapIterator(arrayIterator(moduleSymbols), moduleSymbol => {
             const moduleFileName = moduleSymbol.valueDeclaration.getSourceFile().fileName;
             const sourceDirectory = getDirectoryPath(sourceFile.fileName);