Update TypeScript compiler, linter, and formatter to the latest version

Some formatting tweaks and identifier renaming (to adhere to the
variable unshadowing rule) are necessary.

Closes jquerygh-1906

Author: ariya

package.json

@@ -59,9 +59,9 @@
     "temp": "~0.8.3",
     "test262": "git+https://[email protected]/tc39/test262.git#c4e3d12",
     "test262-stream": "~1.0.0",
-    "tslint": "~5.1.0",
-    "typescript": "~2.3.2",
-    "typescript-formatter": "~5.1.3",
+    "tslint": "~5.8.0",
+    "typescript": "~2.6.2",
+    "typescript-formatter": "~7.0.1",
     "unicode-8.0.0": "~0.7.0",
     "webpack": "~1.14.0"
   },

src/comment-handler.ts

@@ -66,12 +66,12 @@ export class CommentHandler {
             return trailingComments;
         }
 
-        const entry = this.stack[this.stack.length - 1];
-        if (entry && entry.node.trailingComments) {
-            const firstComment = entry.node.trailingComments[0];
+        const last = this.stack[this.stack.length - 1];
+        if (last && last.node.trailingComments) {
+            const firstComment = last.node.trailingComments[0];
             if (firstComment && firstComment.range[0] >= metadata.end.offset) {
-                trailingComments = entry.node.trailingComments;
-                delete entry.node.trailingComments;
+                trailingComments = last.node.trailingComments;
+                delete last.node.trailingComments;
             }
         }
         return trailingComments;

src/jsx-parser.ts

@@ -484,9 +484,9 @@ export class JSXParser extends Parser {
         this.expectJSX('<');
         if (this.matchJSX('/')) {
             this.expectJSX('/');
-            const name = this.parseJSXElementName();
+            const elementName = this.parseJSXElementName();
             this.expectJSX('>');
-            return this.finalize(node, new JSXNode.JSXClosingElement(name));
+            return this.finalize(node, new JSXNode.JSXClosingElement(elementName));
         }
 
         const name = this.parseJSXElementName();

src/scanner.ts

@@ -877,11 +877,11 @@ export class Scanner {
                                 ++this.index;
                                 str += this.scanUnicodeCodePointEscape();
                             } else {
-                                const unescaped = this.scanHexEscape(ch);
-                                if (unescaped === null) {
+                                const unescapedChar = this.scanHexEscape(ch);
+                                if (unescapedChar === null) {
                                     this.throwUnexpectedToken();
                                 }
-                                str += unescaped;
+                                str += unescapedChar;
                             }
                             break;
                         case 'x':
@@ -1003,9 +1003,9 @@ export class Scanner {
                                 cooked += this.scanUnicodeCodePointEscape();
                             } else {
                                 const restore = this.index;
-                                const unescaped = this.scanHexEscape(ch);
-                                if (unescaped !== null) {
-                                    cooked += unescaped;
+                                const unescapedChar = this.scanHexEscape(ch);
+                                if (unescapedChar !== null) {
+                                    cooked += unescapedChar;
                                 } else {
                                     this.index = restore;
                                     cooked += ch;

test/test-262.js

@@ -82,26 +82,26 @@ function report(summary) {
         {
             tests: summary.disallowed.success,
             label:
-            'valid programs parsed without error' +
-            ' (in violation of the whitelist file)'
+                'valid programs parsed without error' +
+                ' (in violation of the whitelist file)'
         },
         {
             tests: summary.disallowed.failure,
             label:
-            'invalid programs produced a parsing error' +
-            ' (in violation of the whitelist file)'
+                'invalid programs produced a parsing error' +
+                ' (in violation of the whitelist file)'
         },
         {
             tests: summary.disallowed.falsePositive,
             label:
-            'invalid programs did not produce a parsing error' +
-            ' (without a corresponding entry in the whitelist file)'
+                'invalid programs did not produce a parsing error' +
+                ' (without a corresponding entry in the whitelist file)'
         },
         {
             tests: summary.disallowed.falseNegative,
             label:
-            'valid programs produced a parsing error' +
-            ' (without a corresponding entry in the whitelist file)'
+                'valid programs produced a parsing error' +
+                ' (without a corresponding entry in the whitelist file)'
         },
         {
             tests: summary.unrecognized,