From b58f16b0213d98c9b2cceea2aa3676d31d92781d Mon Sep 17 00:00:00 2001
From: Cyrus Najmabadi <cyrusn@microsoft.com>
Date: Sun, 1 Mar 2015 11:35:31 -0800
Subject: [PATCH] Don't intern all strings and numbers.  Just the ones used as
 declaration names.

---
 src/services/services.ts | 18 +++++++++++++++++-
 1 file changed, 17 insertions(+), 1 deletion(-)

diff --git a/src/services/services.ts b/src/services/services.ts
index ececc1082ca..a4f1bb8563a 100644
--- a/src/services/services.ts
+++ b/src/services/services.ts
@@ -4044,7 +4044,16 @@ module ts {
                         break;
                     case SyntaxKind.StringLiteral:
                     case SyntaxKind.NumericLiteral:
-                        nameTable[(<LiteralExpression>node).text] = (<LiteralExpression>node).text;
+                        // We want to store any numbers/strings if they were a name that could be
+                        // related to a declaration.  So, if we have 'import x = require("something")'
+                        // then we want 'something' to be in the name table.  Similarly, if we have
+                        // "a['propname']" then we want to store "propname" in the name table.
+                        if (isDeclarationName(node) ||
+                            node.parent.kind === SyntaxKind.ExternalModuleReference ||
+                            isArgumentOfElementAccessExpression(node)) {
+
+                            nameTable[(<LiteralExpression>node).text] = (<LiteralExpression>node).text;
+                        }
                         break;
                     default:
                         forEachChild(node, walk);
@@ -4052,6 +4061,13 @@ module ts {
             } 
         }
 
+        function isArgumentOfElementAccessExpression(node: Node) {
+            return node &&
+                node.parent &&
+                node.parent.kind === SyntaxKind.ElementAccessExpression &&
+                (<ElementAccessExpression>node.parent).argumentExpression === node;
+        }
+
         function getReferencesForNode(node: Node, sourceFiles: SourceFile[], searchOnlyInCurrentFile: boolean, findInStrings: boolean, findInComments: boolean): ReferenceEntry[] {
             // Labels
             if (isLabelName(node)) {