From a6fe980d19c93b8763b00475a1b3e120cc0f3aca Mon Sep 17 00:00:00 2001
From: Cyrus Najmabadi <cyrusn@microsoft.com>
Date: Tue, 2 Dec 2014 13:54:44 -0800
Subject: [PATCH] Add explanatory comment.

Reduce function allocations in the parser.
---
 src/compiler/parser.ts | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

diff --git a/src/compiler/parser.ts b/src/compiler/parser.ts
index 9397661839d..de2d75286dc 100644
--- a/src/compiler/parser.ts
+++ b/src/compiler/parser.ts
@@ -1395,10 +1395,12 @@ module ts {
         }
         
         function parseContextualModifier(t: SyntaxKind): boolean {
-            return token === t && tryParse(() => {
-                nextToken();
-                return canFollowModifier();
-            });
+            return token === t && tryParse(nextTokenCanFollowModifier);
+        }
+
+        function nextTokenCanFollowModifier() {
+            nextToken();
+            return canFollowModifier();
         }
 
         function parseAnyContextualModifier(): boolean {
@@ -4066,9 +4068,13 @@ module ts {
             // literals.  We check to ensure that it is only a string literal later in the grammar
             // walker.
             node.expression = parseExpression();
+
+            // Ensure the string being required is in our 'identifier' table.  This will ensure 
+            // that features like 'find refs' will look inside this file when search for its name.
             if (node.expression.kind === SyntaxKind.StringLiteral) {
                 internIdentifier((<LiteralExpression>node.expression).text);
             }
+
             parseExpected(SyntaxKind.CloseParenToken);
             return finishNode(node);
         }