Added missing semicolons

DickvdBrink · DickvdBrink · commit e0f738429fc4 · 2014-08-07T21:08:38.000+02:00
diff --git a/src/compiler/core.ts b/src/compiler/core.ts
@@ -174,7 +174,7 @@ module ts {
         var result: Map<T> = {};
 
         forEach(array, value => {
-            result[makeKey(value)] = value
+            result[makeKey(value)] = value;
         });
 
         return result;
diff --git a/src/compiler/tc.ts b/src/compiler/tc.ts
@@ -362,8 +362,8 @@ module ts {
         var output = "";
 
         // We want to align our "syntax" and "examples" commands to a certain margin.
-        var syntaxLength = getDiagnosticText(Diagnostics.Syntax_Colon_0, "").length
-        var examplesLength = getDiagnosticText(Diagnostics.Examples_Colon_0, "").length
+        var syntaxLength = getDiagnosticText(Diagnostics.Syntax_Colon_0, "").length;
+        var examplesLength = getDiagnosticText(Diagnostics.Examples_Colon_0, "").length;
         var marginLength = Math.max(syntaxLength, examplesLength);
 
         // Build up the syntactic skeleton.
diff --git a/src/compiler/types.ts b/src/compiler/types.ts
@@ -635,8 +635,8 @@ module ts {
 
     export interface SymbolAccessiblityResult {
         accessibility: SymbolAccessibility;
-        errorSymbolName?: string // Optional symbol name that results in error
-        errorModuleName?: string // If the symbol is not visibile from module, module's name
+        errorSymbolName?: string; // Optional symbol name that results in error
+        errorModuleName?: string; // If the symbol is not visibile from module, module's name
     }
 
     export interface EmitResolver {
diff --git a/src/harness/compilerRunner.ts b/src/harness/compilerRunner.ts
@@ -365,7 +365,7 @@ class CompilerBaselineRunner extends RunnerBase {
                             walker.results.forEach(result => {
                                 var formattedLine = result.identifierName + " : " + result.type;
                                 if (!typeMap[file.unitName]) {
-                                    typeMap[file.unitName] = {}
+                                    typeMap[file.unitName] = {};
                                 }
 
                                 var typeInfo = [formattedLine];
@@ -386,7 +386,7 @@ class CompilerBaselineRunner extends RunnerBase {
                                 walker.results.forEach(result => {
                                     var formattedLine = result.identifierName + " : " + result.type;
                                     if (!typeMap[file.unitName]) {
-                                        typeMap[file.unitName] = {}
+                                        typeMap[file.unitName] = {};
                                     } else {
                                         typeLines.push('No type information for this code.');
                                     }
diff --git a/src/harness/exec.ts b/src/harness/exec.ts
@@ -32,7 +32,7 @@ class WindowsScriptHostExec implements IExec {
             var process = shell.Exec(fileName + ' ' + cmdLineArgs.join(' '));
         } catch(e) {
             result.stderr = e.message;
-            result.exitCode = 1
+            result.exitCode = 1;
             handleResult(result);
             return;
         }
diff --git a/src/harness/fourslash.ts b/src/harness/fourslash.ts
@@ -466,7 +466,7 @@ module FourSlash {
                 this.assertItemInCompletionList(members.entries, symbol, type, docComment, fullSymbolName, kind);
             }
             else {
-                throw new Error("Expected a member list, but none was provided")
+                throw new Error("Expected a member list, but none was provided");
             }
         }
 
@@ -2040,7 +2040,7 @@ module FourSlash {
             globalOptions: opts,
             files: files,
             ranges: ranges
-        }
+        };
     }
 
     enum State {
diff --git a/src/harness/harness.ts b/src/harness/harness.ts
@@ -107,7 +107,7 @@ module Utils {
             } else {
                 return cache[key] = f.apply(this, arguments);
             }
-        })
+        });
     }
 }
 
@@ -555,7 +555,7 @@ module Harness {
                 getCanonicalFileName: ts.getCanonicalFileName,
                 useCaseSensitiveFileNames: () => sys.useCaseSensitiveFileNames,
                 getNewLine: ()=> sys.newLine
-            }
+            };
         }
 
         export class HarnessCompiler {
@@ -1018,7 +1018,7 @@ module Harness {
             }
         }
 
-        var fileCache: { [idx: string]: boolean } = {}
+        var fileCache: { [idx: string]: boolean } = {};
         function generateActual(actualFilename: string, generateContent: () => string): string {
             // For now this is written using TypeScript, because sys is not available when running old test cases.
             // But we need to move to sys once we have
@@ -1083,8 +1083,8 @@ module Harness {
             var lineEndingSensitive = opts && opts.LineEndingSensitive;
 
             if (!lineEndingSensitive) {
-                expected = expected.replace(/\r\n?/g, '\n')
-                actual = actual.replace(/\r\n?/g, '\n')
+                expected = expected.replace(/\r\n?/g, '\n');
+                actual = actual.replace(/\r\n?/g, '\n');
             }
 
             return { expected: expected, actual: actual };
diff --git a/src/harness/harnessLanguageService.ts b/src/harness/harnessLanguageService.ts
@@ -91,7 +91,7 @@ module Harness.LanguageService {
     }
 
     class CancellationToken {
-        public static None: CancellationToken = new CancellationToken(null)
+        public static None: CancellationToken = new CancellationToken(null);
 
         constructor(private cancellationToken: ts.CancellationToken) {
         }
@@ -283,7 +283,7 @@ module Harness.LanguageService {
 
         /** Parse a file on disk given its fileName */
         public parseFile(fileName: string) {
-            var sourceText = TypeScript.ScriptSnapshot.fromString(Harness.IO.readFile(fileName))
+            var sourceText = TypeScript.ScriptSnapshot.fromString(Harness.IO.readFile(fileName));
             return this.parseSourceText(fileName, sourceText);
         }
 
diff --git a/src/harness/runner.ts b/src/harness/runner.ts
@@ -41,7 +41,7 @@ var mytestconfig = 'mytest.config';
 var testconfig = 'test.config';
 var testConfigFile =
     Harness.IO.fileExists(mytestconfig) ? Harness.IO.readFile(mytestconfig) :
-    (Harness.IO.fileExists(testconfig) ? Harness.IO.readFile(testconfig) : '')
+    (Harness.IO.fileExists(testconfig) ? Harness.IO.readFile(testconfig) : '');
 
 if (testConfigFile !== '') {
     // TODO: not sure why this is crashing mocha
diff --git a/src/services/braceMatcher.ts b/src/services/braceMatcher.ts
@@ -36,7 +36,7 @@ module TypeScript.Services {
             if (start(currentToken) === position) {
                 var closingBraceKind = BraceMatcher.getMatchingCloseBraceTokenKind(currentToken);
                 if (closingBraceKind !== null) {
-                    var parentElement = currentToken.parent
+                    var parentElement = currentToken.parent;
                     var currentPosition = fullStart(currentToken.parent);
                     for (var i = 0, n = childCount(parentElement); i < n; i++) {
                         var element = childAt(parentElement, i);
@@ -87,7 +87,7 @@ module TypeScript.Services {
             var element = positionedElement !== null && positionedElement;
             switch (element.kind()) {
                 case TypeScript.SyntaxKind.OpenBraceToken:
-                    return TypeScript.SyntaxKind.CloseBraceToken
+                    return TypeScript.SyntaxKind.CloseBraceToken;
                 case TypeScript.SyntaxKind.OpenParenToken:
                     return TypeScript.SyntaxKind.CloseParenToken;
                 case TypeScript.SyntaxKind.OpenBracketToken:
diff --git a/src/services/compiler/hashTable.ts b/src/services/compiler/hashTable.ts
@@ -16,7 +16,7 @@
 ///<reference path='references.ts' />
 
 module TypeScript {
-    var proto = "__proto__"
+    var proto = "__proto__";
 
     class BlockIntrinsics<T> {
         public prototype: T = undefined;
diff --git a/src/services/compiler/optionsParser.ts b/src/services/compiler/optionsParser.ts
@@ -155,7 +155,7 @@ module TypeScript {
 
             config.name = name;
             config.short = short;
-            config.flag = true
+            config.flag = true;
 
         this.options.push(config);
         }
diff --git a/src/services/compiler/precompile.ts b/src/services/compiler/precompile.ts
@@ -150,7 +150,7 @@ module TypeScript {
         var noDefaultLib = false;
         var diagnostics: Diagnostic[] = [];
         var referencedFiles: IFileReference[] = [];
-        var lineMap = text.lineMap()
+        var lineMap = text.lineMap();
 
         for (var i = 0, n = leadingTrivia.count(); i < n; i++) {
             var trivia = leadingTrivia.syntaxTriviaAt(i);
diff --git a/src/services/compiler/sourceMapping.ts b/src/services/compiler/sourceMapping.ts
@@ -94,9 +94,9 @@ module TypeScript {
             var expectedAst = this.mappingLevel.pop();
             if (ast !== expectedAst) {
                 var expectedAstInfo: any = (<any>expectedAst).kind ? SyntaxKind[(<any>expectedAst).kind] : [expectedAst.start(), expectedAst.end()];
-                var astInfo: any = (<any>ast).kind ? SyntaxKind[(<any>ast).kind] : [ast.start(), ast.end()]
+                var astInfo: any = (<any>ast).kind ? SyntaxKind[(<any>ast).kind] : [ast.start(), ast.end()];
                 Debug.fail(
-                    "Provided ast is not the expected ISyntaxElement, Expected: " + expectedAstInfo + " Given: " + astInfo)
+                    "Provided ast is not the expected ISyntaxElement, Expected: " + expectedAstInfo + " Given: " + astInfo);
 
             }
         }
diff --git a/src/services/compiler/typescript.ts b/src/services/compiler/typescript.ts
@@ -131,7 +131,7 @@ module TypeScript {
 
     export class CancellationToken {
 
-        public static None: CancellationToken = new CancellationToken(null)
+        public static None: CancellationToken = new CancellationToken(null);
 
         constructor(private cancellationToken: ICancellationToken) {
         }
@@ -237,7 +237,7 @@ module TypeScript {
                     };
                 });
                 documents.sort((x, y) => y.refCount - x.refCount);
-                return { bucket: name, documents: documents }
+                return { bucket: name, documents: documents };
             });
             return JSON.stringify(bucketInfoArray, null, 2);
         }
@@ -705,7 +705,7 @@ module TypeScript {
         //
 
         public getSyntacticDiagnostics(fileName: string): Diagnostic[] {
-            fileName = TypeScript.switchToForwardSlashes(fileName)
+            fileName = TypeScript.switchToForwardSlashes(fileName);
             return this.getDocument(fileName).diagnostics();
         }
 
@@ -724,7 +724,7 @@ module TypeScript {
             var document = this.getDocument(fileName);
 
             var startTime = (new Date()).getTime();
-            PullTypeResolver.typeCheck(this.compilationSettings(), this.semanticInfoChain, document)
+            PullTypeResolver.typeCheck(this.compilationSettings(), this.semanticInfoChain, document);
             var endTime = (new Date()).getTime();
 
             typeCheckTime += endTime - startTime;
@@ -1464,7 +1464,7 @@ module TypeScript {
                 // shared emitter (and we'll take care of it after all the files are done.
                 this._sharedEmitter = this.compiler._emitDocument(
                     document, this._emitOptions,
-                    outputFiles => { this._current = CompileResult.fromOutputFiles(outputFiles) },
+                    outputFiles => { this._current = CompileResult.fromOutputFiles(outputFiles); },
                     this._sharedEmitter);
                 return true;
             }
diff --git a/src/services/findReferenceHelpers.ts b/src/services/findReferenceHelpers.ts
@@ -50,7 +50,7 @@ module TypeScript.Services {
                     var secondSymbolDecl = secondSymbol.getDeclarations()[0];
 
                     var firstParentDecl = firstSymbolDecl.getParentDecl();
-                    var secondParentDecl = secondSymbolDecl.getParentDecl()
+                    var secondParentDecl = secondSymbolDecl.getParentDecl();
 
                     if (firstParentDecl.kind === TypeScript.PullElementKind.ObjectLiteral &&
                         secondParentDecl.kind === TypeScript.PullElementKind.ObjectLiteral) {
diff --git a/src/services/formatting/formattingManager.ts b/src/services/formatting/formattingManager.ts
@@ -22,7 +22,7 @@ module TypeScript.Services.Formatting {
         constructor(private syntaxTree: SyntaxTree, private snapshot: ITextSnapshot, private rulesProvider: RulesProvider, editorOptions: ts.EditorOptions) {
             //
             // TODO: convert to use FormattingOptions instead of EditorOptions
-            this.options = new FormattingOptions(!editorOptions.ConvertTabsToSpaces, editorOptions.TabSize, editorOptions.IndentSize, editorOptions.NewLineCharacter)
+            this.options = new FormattingOptions(!editorOptions.ConvertTabsToSpaces, editorOptions.TabSize, editorOptions.IndentSize, editorOptions.NewLineCharacter);
         }
 
         public formatSelection(minChar: number, limChar: number): ts.TextEdit[] {
diff --git a/src/services/formatting/ruleDescriptor.ts b/src/services/formatting/ruleDescriptor.ts
@@ -26,7 +26,7 @@ module TypeScript.Services.Formatting {
         }
 
         static create1(left: SyntaxKind, right: SyntaxKind): RuleDescriptor {
-            return RuleDescriptor.create4(Shared.TokenRange.FromToken(left), Shared.TokenRange.FromToken(right))
+            return RuleDescriptor.create4(Shared.TokenRange.FromToken(left), Shared.TokenRange.FromToken(right));
         }
 
         static create2(left: Shared.TokenRange, right: SyntaxKind): RuleDescriptor {

Original file line number	Diff line number	Diff line change
`@@ -635,8 +635,8 @@ module ts {`
`635`	`635`
`636`	`636`	`export interface SymbolAccessiblityResult {`
`637`	`637`	`accessibility: SymbolAccessibility;`
`638`		`- errorSymbolName?: string // Optional symbol name that results in error`
`639`		`- errorModuleName?: string // If the symbol is not visibile from module, module's name`
	`638`	`+ errorSymbolName?: string; // Optional symbol name that results in error`
	`639`	`+ errorModuleName?: string; // If the symbol is not visibile from module, module's name`
`640`	`640`	`}`
`641`	`641`
`642`	`642`	`export interface EmitResolver {`
Original file line number	Diff line number	Diff line change
`@@ -32,7 +32,7 @@ class WindowsScriptHostExec implements IExec {`
`32`	`32`	`var process = shell.Exec(fileName + ' ' + cmdLineArgs.join(' '));`
`33`	`33`	`} catch(e) {`
`34`	`34`	`result.stderr = e.message;`
`35`		`- result.exitCode = 1`
	`35`	`+ result.exitCode = 1;`
`36`	`36`	`handleResult(result);`
`37`	`37`	`return;`
`38`	`38`	`}`
Original file line number	Diff line number	Diff line change
`@@ -466,7 +466,7 @@ module FourSlash {`
`466`	`466`	`this.assertItemInCompletionList(members.entries, symbol, type, docComment, fullSymbolName, kind);`
`467`	`467`	`}`
`468`	`468`	`else {`
`469`		`- throw new Error("Expected a member list, but none was provided")`
	`469`	`+ throw new Error("Expected a member list, but none was provided");`
`470`	`470`	`}`
`471`	`471`	`}`
`472`	`472`
`@@ -2040,7 +2040,7 @@ module FourSlash {`
`2040`	`2040`	`globalOptions: opts,`
`2041`	`2041`	`files: files,`
`2042`	`2042`	`ranges: ranges`
`2043`		`- }`
	`2043`	`+ };`
`2044`	`2044`	`}`
`2045`	`2045`
`2046`	`2046`	`enum State {`
Original file line number	Diff line number	Diff line change
`@@ -107,7 +107,7 @@ module Utils {`
`107`	`107`	`} else {`
`108`	`108`	`return cache[key] = f.apply(this, arguments);`
`109`	`109`	`}`
`110`		`- })`
	`110`	`+ });`
`111`	`111`	`}`
`112`	`112`	`}`
`113`	`113`
`@@ -555,7 +555,7 @@ module Harness {`
`555`	`555`	`getCanonicalFileName: ts.getCanonicalFileName,`
`556`	`556`	`useCaseSensitiveFileNames: () => sys.useCaseSensitiveFileNames,`
`557`	`557`	`getNewLine: ()=> sys.newLine`
`558`		`- }`
	`558`	`+ };`
`559`	`559`	`}`
`560`	`560`
`561`	`561`	`export class HarnessCompiler {`
`@@ -1018,7 +1018,7 @@ module Harness {`
`1018`	`1018`	`}`
`1019`	`1019`	`}`
`1020`	`1020`
`1021`		`- var fileCache: { [idx: string]: boolean } = {}`
	`1021`	`+ var fileCache: { [idx: string]: boolean } = {};`
`1022`	`1022`	`function generateActual(actualFilename: string, generateContent: () => string): string {`
`1023`	`1023`	`// For now this is written using TypeScript, because sys is not available when running old test cases.`
`1024`	`1024`	`// But we need to move to sys once we have`
`@@ -1083,8 +1083,8 @@ module Harness {`
`1083`	`1083`	`var lineEndingSensitive = opts && opts.LineEndingSensitive;`
`1084`	`1084`
`1085`	`1085`	`if (!lineEndingSensitive) {`
`1086`		`- expected = expected.replace(/\r\n?/g, '\n')`
`1087`		`- actual = actual.replace(/\r\n?/g, '\n')`
	`1086`	`+ expected = expected.replace(/\r\n?/g, '\n');`
	`1087`	`+ actual = actual.replace(/\r\n?/g, '\n');`
`1088`	`1088`	`}`
`1089`	`1089`
`1090`	`1090`	`return { expected: expected, actual: actual };`
Original file line number	Diff line number	Diff line change
`@@ -91,7 +91,7 @@ module Harness.LanguageService {`
`91`	`91`	`}`
`92`	`92`
`93`	`93`	`class CancellationToken {`
`94`		`- public static None: CancellationToken = new CancellationToken(null)`
	`94`	`+ public static None: CancellationToken = new CancellationToken(null);`
`95`	`95`
`96`	`96`	`constructor(private cancellationToken: ts.CancellationToken) {`
`97`	`97`	`}`
`@@ -283,7 +283,7 @@ module Harness.LanguageService {`
`283`	`283`
`284`	`284`	`/** Parse a file on disk given its fileName */`
`285`	`285`	`public parseFile(fileName: string) {`
`286`		`- var sourceText = TypeScript.ScriptSnapshot.fromString(Harness.IO.readFile(fileName))`
	`286`	`+ var sourceText = TypeScript.ScriptSnapshot.fromString(Harness.IO.readFile(fileName));`
`287`	`287`	`return this.parseSourceText(fileName, sourceText);`
`288`	`288`	`}`
`289`	`289`