Addressed CR feedback.

DanielRosenwasser · DanielRosenwasser · commit d353d624346d · 2015-07-07T12:48:17.000-07:00
diff --git a/src/services/services.ts b/src/services/services.ts
@@ -3204,6 +3204,12 @@ namespace ts {
                 return false;
             }
 
+            /**
+             * Aggregates relevant symbols for completion in object literals and object binding patterns.
+             * Relevant symbols are stored in the captured 'symbols' variable.
+             *
+             * @returns true if 'symbols' was successfully populated; false otherwise.
+             */
             function tryGetObjectLikeCompletionSymbols(objectLikeContainer: ObjectLiteralExpression | BindingPattern): boolean {
                 // We're looking up possible property names from contextual/inferred/declared type.
                 isMemberCompletion = true;
@@ -3242,6 +3248,15 @@ namespace ts {
                 return true;
             }
 
+            /**
+             * Aggregates relevant symbols for completion in import clauses; for instance,
+             *
+             *      import { $ } from "moduleName";
+             *
+             * Relevant symbols are stored in the captured 'symbols' variable.
+             *
+             * @returns true if 'symbols' was successfully populated; false otherwise.
+             */
             function tryGetImportClauseCompletionSymbols(importClause: ImportClause): boolean {
                 // cursor is in import clause
                 // try to show exported member for imported module
diff --git a/tests/cases/fourslash/completionListInImportClause01.ts b/tests/cases/fourslash/completionListInImportClause01.ts
@@ -14,8 +14,8 @@
 function verifyCompletionAtMarker(marker: string, showBuilder: boolean, ...completions: string[]) {
     goTo.marker(marker);
     if (completions.length) {
-        for (let i = 0; i < completions.length; ++i) {
-            verify.completionListContains(completions[i]);
+        for (let completion of completions) {
+            verify.completionListContains(completion);
         }
     }
     else {

Original file line number	Diff line number	Diff line change
`@@ -14,8 +14,8 @@`
`14`	`14`	`function verifyCompletionAtMarker(marker: string, showBuilder: boolean, ...completions: string[]) {`
`15`	`15`	`goTo.marker(marker);`
`16`	`16`	`if (completions.length) {`
`17`		`- for (let i = 0; i < completions.length; ++i) {`
`18`		`- verify.completionListContains(completions[i]);`
	`17`	`+ for (let completion of completions) {`
	`18`	`+ verify.completionListContains(completion);`
`19`	`19`	`}`
`20`	`20`	`}`
`21`	`21`	`else {`