Merge pull request #10757 from Microsoft/Fix10755

Fix #10755: remove uses of `package` as an identifier name
microsoft · Sep 7, 2016 · c16ae3e · c16ae3e
2 parents 5c72a32 + 9ee897a
commit c16ae3e
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/src/services/services.ts b/src/services/services.ts
@@ -4048,8 +4048,8 @@ namespace ts {
 
                 if (host.getDirectories) {
                     // Also get all @types typings installed in visible node_modules directories
-                    for (const package of findPackageJsons(scriptPath)) {
-                        const typesDir = combinePaths(getDirectoryPath(package), "node_modules/@types");
+                    for (const packageJson of findPackageJsons(scriptPath)) {
+                        const typesDir = combinePaths(getDirectoryPath(packageJson), "node_modules/@types");
                         getCompletionEntriesFromDirectories(host, options, typesDir, span, result);
                     }
                 }
@@ -4095,8 +4095,8 @@ namespace ts {
 
                 if (host.readFile && host.fileExists) {
                     for (const packageJson of findPackageJsons(scriptPath)) {
-                        const package = tryReadingPackageJson(packageJson);
-                        if (!package) {
+                        const contents = tryReadingPackageJson(packageJson);
+                        if (!contents) {
                             return;
                         }
 
@@ -4105,7 +4105,7 @@ namespace ts {
 
                         // Provide completions for all non @types dependencies
                         for (const key of nodeModulesDependencyKeys) {
-                            addPotentialPackageNames(package[key], foundModuleNames);
+                            addPotentialPackageNames(contents[key], foundModuleNames);
                         }
 
                         for (const moduleName of foundModuleNames) {