Fix comments

mhegazy · mhegazy · commit 8b8d33d8b804 · 2015-10-16T16:29:11.000-07:00
diff --git a/tests/baselines/reference/augmentedTypesClass2.errors.txt b/tests/baselines/reference/augmentedTypesClass2.errors.txt
@@ -6,13 +6,13 @@ tests/cases/compiler/augmentedTypesClass2.ts(21,6): error TS2300: Duplicate iden
     // Checking class with other things in type space not value space
     
     // class then interface
-    class c11 { // error
+    class c11 {
         foo() {
             return 1;
         }
     }
     
-    interface c11 { // error
+    interface c11 {
         bar(): void;
     }
     
diff --git a/tests/baselines/reference/augmentedTypesClass2.js b/tests/baselines/reference/augmentedTypesClass2.js
@@ -2,13 +2,13 @@
 // Checking class with other things in type space not value space
 
 // class then interface
-class c11 { // error
+class c11 {
     foo() {
         return 1;
     }
 }
 
-interface c11 { // error
+interface c11 {
     bar(): void;
 }
 
diff --git a/tests/baselines/reference/augmentedTypesInterface.errors.txt b/tests/baselines/reference/augmentedTypesInterface.errors.txt
@@ -14,11 +14,11 @@ tests/cases/compiler/augmentedTypesInterface.ts(26,6): error TS2300: Duplicate i
     }
     
     // interface then class
-    interface i2 { // error
+    interface i2 {
         foo(): void;
     }
     
-    class i2 {  // error
+    class i2 {
         bar() {
             return 1;
         }
diff --git a/tests/baselines/reference/augmentedTypesInterface.js b/tests/baselines/reference/augmentedTypesInterface.js
@@ -10,11 +10,11 @@ interface i {
 }
 
 // interface then class
-interface i2 { // error
+interface i2 {
     foo(): void;
 }
 
-class i2 {  // error
+class i2 {
     bar() {
         return 1;
     }
diff --git a/tests/baselines/reference/classAndInterfaceWithSameName.errors.txt b/tests/baselines/reference/classAndInterfaceWithSameName.errors.txt
@@ -8,7 +8,7 @@ tests/cases/conformance/classes/classDeclarations/classAndInterfaceWithSameName.
     class C { foo: string; }
               ~~~
 !!! error TS2300: Duplicate identifier 'foo'.
-    interface C { foo: string; } // error
+    interface C { foo: string; }
                   ~~~
 !!! error TS2300: Duplicate identifier 'foo'.
     
@@ -19,7 +19,7 @@ tests/cases/conformance/classes/classDeclarations/classAndInterfaceWithSameName.
 !!! error TS2300: Duplicate identifier 'bar'.
         }
     
-        interface D { // error
+        interface D {
             bar: string;
             ~~~
 !!! error TS2300: Duplicate identifier 'bar'.
diff --git a/tests/baselines/reference/classAndInterfaceWithSameName.js b/tests/baselines/reference/classAndInterfaceWithSameName.js
@@ -1,13 +1,13 @@
 //// [classAndInterfaceWithSameName.ts]
 class C { foo: string; }
-interface C { foo: string; } // error
+interface C { foo: string; }
 
 module M {
     class D {
         bar: string;
     }
 
-    interface D { // error
+    interface D {
         bar: string;
     }
 }
diff --git a/tests/baselines/reference/duplicateIdentifiersAcrossContainerBoundaries.errors.txt b/tests/baselines/reference/duplicateIdentifiersAcrossContainerBoundaries.errors.txt
@@ -9,7 +9,7 @@ tests/cases/compiler/duplicateIdentifiersAcrossContainerBoundaries.ts(41,16): er
         export interface I { }
     }
     module M {
-        export class I { } // error
+        export class I { }
     }
     
     module M {
diff --git a/tests/baselines/reference/duplicateIdentifiersAcrossContainerBoundaries.js b/tests/baselines/reference/duplicateIdentifiersAcrossContainerBoundaries.js
@@ -3,7 +3,7 @@ module M {
     export interface I { }
 }
 module M {
-    export class I { } // error
+    export class I { }
 }
 
 module M {
@@ -60,7 +60,7 @@ var M;
         }
         return I;
     })();
-    M.I = I; // error
+    M.I = I;
 })(M || (M = {}));
 var M;
 (function (M) {
diff --git a/tests/baselines/reference/interfaceClassMerging2.js b/tests/baselines/reference/interfaceClassMerging2.js
@@ -28,7 +28,7 @@ class Bar extends Foo {
 
 
 var bar = new Bar();
-bar.classFooMethod().interfaceFooMethod().classBarMethod().classFooMethod();
+bar.interfaceBarMethod().interfaceFooMethod().classBarMethod().classFooMethod();
 
 
 var foo = new Foo();
@@ -61,6 +61,6 @@ var Bar = (function (_super) {
     return Bar;
 })(Foo);
 var bar = new Bar();
-bar.classFooMethod().interfaceFooMethod().classBarMethod().classFooMethod();
+bar.interfaceBarMethod().interfaceFooMethod().classBarMethod().classFooMethod();
 var foo = new Foo();
 foo = bar;
diff --git a/tests/baselines/reference/interfaceClassMerging2.symbols b/tests/baselines/reference/interfaceClassMerging2.symbols
@@ -54,13 +54,13 @@ var bar = new Bar();
 >bar : Symbol(bar, Decl(interfaceClassMerging2.ts, 28, 3))
 >Bar : Symbol(Bar, Decl(interfaceClassMerging2.ts, 11, 1), Decl(interfaceClassMerging2.ts, 17, 1))
 
-bar.classFooMethod().interfaceFooMethod().classBarMethod().classFooMethod();
->bar.classFooMethod().interfaceFooMethod().classBarMethod().classFooMethod : Symbol(Foo.classFooMethod, Decl(interfaceClassMerging2.ts, 6, 27))
->bar.classFooMethod().interfaceFooMethod().classBarMethod : Symbol(Bar.classBarMethod, Decl(interfaceClassMerging2.ts, 20, 27))
->bar.classFooMethod().interfaceFooMethod : Symbol(Foo.interfaceFooMethod, Decl(interfaceClassMerging2.ts, 0, 15))
->bar.classFooMethod : Symbol(Foo.classFooMethod, Decl(interfaceClassMerging2.ts, 6, 27))
+bar.interfaceBarMethod().interfaceFooMethod().classBarMethod().classFooMethod();
+>bar.interfaceBarMethod().interfaceFooMethod().classBarMethod().classFooMethod : Symbol(Foo.classFooMethod, Decl(interfaceClassMerging2.ts, 6, 27))
+>bar.interfaceBarMethod().interfaceFooMethod().classBarMethod : Symbol(Bar.classBarMethod, Decl(interfaceClassMerging2.ts, 20, 27))
+>bar.interfaceBarMethod().interfaceFooMethod : Symbol(Foo.interfaceFooMethod, Decl(interfaceClassMerging2.ts, 0, 15))
+>bar.interfaceBarMethod : Symbol(Bar.interfaceBarMethod, Decl(interfaceClassMerging2.ts, 14, 15))
 >bar : Symbol(bar, Decl(interfaceClassMerging2.ts, 28, 3))
->classFooMethod : Symbol(Foo.classFooMethod, Decl(interfaceClassMerging2.ts, 6, 27))
+>interfaceBarMethod : Symbol(Bar.interfaceBarMethod, Decl(interfaceClassMerging2.ts, 14, 15))
 >interfaceFooMethod : Symbol(Foo.interfaceFooMethod, Decl(interfaceClassMerging2.ts, 0, 15))
 >classBarMethod : Symbol(Bar.classBarMethod, Decl(interfaceClassMerging2.ts, 20, 27))
 >classFooMethod : Symbol(Foo.classFooMethod, Decl(interfaceClassMerging2.ts, 6, 27))
diff --git a/tests/baselines/reference/interfaceClassMerging2.types b/tests/baselines/reference/interfaceClassMerging2.types
@@ -55,17 +55,17 @@ var bar = new Bar();
 >new Bar() : Bar
 >Bar : typeof Bar
 
-bar.classFooMethod().interfaceFooMethod().classBarMethod().classFooMethod();
->bar.classFooMethod().interfaceFooMethod().classBarMethod().classFooMethod() : Bar
->bar.classFooMethod().interfaceFooMethod().classBarMethod().classFooMethod : () => Bar
->bar.classFooMethod().interfaceFooMethod().classBarMethod() : Bar
->bar.classFooMethod().interfaceFooMethod().classBarMethod : () => Bar
->bar.classFooMethod().interfaceFooMethod() : Bar
->bar.classFooMethod().interfaceFooMethod : () => Bar
->bar.classFooMethod() : Bar
->bar.classFooMethod : () => Bar
+bar.interfaceBarMethod().interfaceFooMethod().classBarMethod().classFooMethod();
+>bar.interfaceBarMethod().interfaceFooMethod().classBarMethod().classFooMethod() : Bar
+>bar.interfaceBarMethod().interfaceFooMethod().classBarMethod().classFooMethod : () => Bar
+>bar.interfaceBarMethod().interfaceFooMethod().classBarMethod() : Bar
+>bar.interfaceBarMethod().interfaceFooMethod().classBarMethod : () => Bar
+>bar.interfaceBarMethod().interfaceFooMethod() : Bar
+>bar.interfaceBarMethod().interfaceFooMethod : () => Bar
+>bar.interfaceBarMethod() : Bar
+>bar.interfaceBarMethod : () => Bar
 >bar : Bar
->classFooMethod : () => Bar
+>interfaceBarMethod : () => Bar
 >interfaceFooMethod : () => Bar
 >classBarMethod : () => Bar
 >classFooMethod : () => Bar
diff --git a/tests/baselines/reference/mergedClassInterface.js b/tests/baselines/reference/mergedClassInterface.js
@@ -11,13 +11,13 @@ interface C2 { }
 
 declare class C2 { }
 
-class C3 { } // error -- cannot merge non-ambient class and interface
+class C3 { }
 
-interface C3 { } // error -- cannot merge non-ambient class and interface
+interface C3 { }
 
-interface C4 { } // error -- cannot merge non-ambient class and interface
+interface C4 { }
 
-class C4 { } // error -- cannot merge non-ambient class and interface
+class C4 { }
 
 interface C5 {
     x1: number;
@@ -59,12 +59,12 @@ var C3 = (function () {
     function C3() {
     }
     return C3;
-})(); // error -- cannot merge non-ambient class and interface
+})();
 var C4 = (function () {
     function C4() {
     }
     return C4;
-})(); // error -- cannot merge non-ambient class and interface
+})();
 // checks if properties actually were merged
 var c5;
 c5.x1;
diff --git a/tests/baselines/reference/mergedClassInterface.symbols b/tests/baselines/reference/mergedClassInterface.symbols
@@ -13,16 +13,16 @@ interface C2 { }
 declare class C2 { }
 >C2 : Symbol(C2, Decl(file1.ts, 4, 16), Decl(file1.ts, 6, 16))
 
-class C3 { } // error -- cannot merge non-ambient class and interface
+class C3 { }
 >C3 : Symbol(C3, Decl(file1.ts, 8, 20), Decl(file1.ts, 10, 12))
 
-interface C3 { } // error -- cannot merge non-ambient class and interface
+interface C3 { }
 >C3 : Symbol(C3, Decl(file1.ts, 8, 20), Decl(file1.ts, 10, 12))
 
-interface C4 { } // error -- cannot merge non-ambient class and interface
+interface C4 { }
 >C4 : Symbol(C4, Decl(file1.ts, 12, 16), Decl(file1.ts, 14, 16))
 
-class C4 { } // error -- cannot merge non-ambient class and interface
+class C4 { }
 >C4 : Symbol(C4, Decl(file1.ts, 12, 16), Decl(file1.ts, 14, 16))
 
 interface C5 {
diff --git a/tests/baselines/reference/mergedClassInterface.types b/tests/baselines/reference/mergedClassInterface.types
@@ -13,16 +13,16 @@ interface C2 { }
 declare class C2 { }
 >C2 : C2
 
-class C3 { } // error -- cannot merge non-ambient class and interface
+class C3 { }
 >C3 : C3
 
-interface C3 { } // error -- cannot merge non-ambient class and interface
+interface C3 { }
 >C3 : C3
 
-interface C4 { } // error -- cannot merge non-ambient class and interface
+interface C4 { }
 >C4 : C4
 
-class C4 { } // error -- cannot merge non-ambient class and interface
+class C4 { }
 >C4 : C4
 
 interface C5 {
diff --git a/tests/baselines/reference/nameCollisions.errors.txt b/tests/baselines/reference/nameCollisions.errors.txt
@@ -82,8 +82,8 @@ tests/cases/compiler/nameCollisions.ts(37,11): error TS2300: Duplicate identifie
         interface fi { } // ok
     
         class cli { }
-        interface cli { } // error
+        interface cli { }
     
         interface cli2 { }
-        class cli2 { } // error
+        class cli2 { }
     }
diff --git a/tests/baselines/reference/nameCollisions.js b/tests/baselines/reference/nameCollisions.js
@@ -41,10 +41,10 @@ module T {
     interface fi { } // ok
 
     class cli { }
-    interface cli { } // error
+    interface cli { }
 
     interface cli2 { }
-    class cli2 { } // error
+    class cli2 { }
 }
 
 //// [nameCollisions.js]
@@ -102,5 +102,5 @@ var T;
         function cli2() {
         }
         return cli2;
-    })(); // error
+    })();
 })(T || (T = {}));
diff --git a/tests/cases/compiler/augmentedTypesClass2.ts b/tests/cases/compiler/augmentedTypesClass2.ts
@@ -1,13 +1,13 @@
 // Checking class with other things in type space not value space
 
 // class then interface
-class c11 { // error
+class c11 {
     foo() {
         return 1;
     }
 }
 
-interface c11 { // error
+interface c11 {
     bar(): void;
 }
 
diff --git a/tests/cases/compiler/augmentedTypesInterface.ts b/tests/cases/compiler/augmentedTypesInterface.ts
@@ -9,11 +9,11 @@ interface i {
 }
 
 // interface then class
-interface i2 { // error
+interface i2 {
     foo(): void;
 }
 
-class i2 {  // error
+class i2 {
     bar() {
         return 1;
     }
diff --git a/tests/cases/compiler/duplicateIdentifiersAcrossContainerBoundaries.ts b/tests/cases/compiler/duplicateIdentifiersAcrossContainerBoundaries.ts
@@ -2,7 +2,7 @@ module M {
     export interface I { }
 }
 module M {
-    export class I { } // error
+    export class I { }
 }
 
 module M {
diff --git a/tests/cases/compiler/interfaceClassMerging2.ts b/tests/cases/compiler/interfaceClassMerging2.ts
@@ -27,7 +27,7 @@ class Bar extends Foo {
 
 
 var bar = new Bar();
-bar.classFooMethod().interfaceFooMethod().classBarMethod().classFooMethod();
+bar.interfaceBarMethod().interfaceFooMethod().classBarMethod().classFooMethod();
 
 
 var foo = new Foo();
diff --git a/tests/cases/compiler/nameCollisions.ts b/tests/cases/compiler/nameCollisions.ts
@@ -40,8 +40,8 @@ module T {
     interface fi { } // ok
 
     class cli { }
-    interface cli { } // error
+    interface cli { }
 
     interface cli2 { }
-    class cli2 { } // error
+    class cli2 { }
 }
diff --git a/tests/cases/conformance/classes/classDeclarations/classAndInterfaceWithSameName.ts b/tests/cases/conformance/classes/classDeclarations/classAndInterfaceWithSameName.ts
@@ -1,12 +1,12 @@
 class C { foo: string; }
-interface C { foo: string; } // error
+interface C { foo: string; }
 
 module M {
     class D {
         bar: string;
     }
 
-    interface D { // error
+    interface D {
         bar: string;
     }
 }
diff --git a/tests/cases/conformance/classes/classDeclarations/mergedClassInterface.ts b/tests/cases/conformance/classes/classDeclarations/mergedClassInterface.ts
@@ -10,13 +10,13 @@ interface C2 { }
 
 declare class C2 { }
 
-class C3 { } // error -- cannot merge non-ambient class and interface
+class C3 { }
 
-interface C3 { } // error -- cannot merge non-ambient class and interface
+interface C3 { }
 
-interface C4 { } // error -- cannot merge non-ambient class and interface
+interface C4 { }
 
-class C4 { } // error -- cannot merge non-ambient class and interface
+class C4 { }
 
 interface C5 {
     x1: number;

Original file line number	Diff line number	Diff line change
`@@ -6,13 +6,13 @@ tests/cases/compiler/augmentedTypesClass2.ts(21,6): error TS2300: Duplicate iden`
`6`	`6`	`// Checking class with other things in type space not value space`
`7`	`7`
`8`	`8`	`// class then interface`
`9`		`- class c11 { // error`
	`9`	`+ class c11 {`
`10`	`10`	`foo() {`
`11`	`11`	`return 1;`
`12`	`12`	`}`
`13`	`13`	`}`
`14`	`14`
`15`		`- interface c11 { // error`
	`15`	`+ interface c11 {`
`16`	`16`	`bar(): void;`
`17`	`17`	`}`
`18`	`18`
Original file line number	Diff line number	Diff line change
`@@ -2,13 +2,13 @@`
`2`	`2`	`// Checking class with other things in type space not value space`
`3`	`3`
`4`	`4`	`// class then interface`
`5`		`-class c11 { // error`
	`5`	`+class c11 {`
`6`	`6`	`foo() {`
`7`	`7`	`return 1;`
`8`	`8`	`}`
`9`	`9`	`}`
`10`	`10`
`11`		`-interface c11 { // error`
	`11`	`+interface c11 {`
`12`	`12`	`bar(): void;`
`13`	`13`	`}`
`14`	`14`
Original file line number	Diff line number	Diff line change
`@@ -14,11 +14,11 @@ tests/cases/compiler/augmentedTypesInterface.ts(26,6): error TS2300: Duplicate i`
`14`	`14`	`}`
`15`	`15`
`16`	`16`	`// interface then class`
`17`		`- interface i2 { // error`
	`17`	`+ interface i2 {`
`18`	`18`	`foo(): void;`
`19`	`19`	`}`
`20`	`20`
`21`		`- class i2 { // error`
	`21`	`+ class i2 {`
`22`	`22`	`bar() {`
`23`	`23`	`return 1;`
`24`	`24`	`}`
Original file line number	Diff line number	Diff line change
`@@ -10,11 +10,11 @@ interface i {`
`10`	`10`	`}`
`11`	`11`
`12`	`12`	`// interface then class`
`13`		`-interface i2 { // error`
	`13`	`+interface i2 {`
`14`	`14`	`foo(): void;`
`15`	`15`	`}`
`16`	`16`
`17`		`-class i2 { // error`
	`17`	`+class i2 {`
`18`	`18`	`bar() {`
`19`	`19`	`return 1;`
`20`	`20`	`}`
Original file line number	Diff line number	Diff line change
`@@ -1,13 +1,13 @@`
`1`	`1`	`//// [classAndInterfaceWithSameName.ts]`
`2`	`2`	`class C { foo: string; }`
`3`		`-interface C { foo: string; } // error`
	`3`	`+interface C { foo: string; }`
`4`	`4`
`5`	`5`	`module M {`
`6`	`6`	`class D {`
`7`	`7`	`bar: string;`
`8`	`8`	`}`
`9`	`9`
`10`		`- interface D { // error`
	`10`	`+ interface D {`
`11`	`11`	`bar: string;`
`12`	`12`	`}`
`13`	`13`	`}`
Original file line number	Diff line number	Diff line change
`@@ -9,7 +9,7 @@ tests/cases/compiler/duplicateIdentifiersAcrossContainerBoundaries.ts(41,16): er`
`9`	`9`	`export interface I { }`
`10`	`10`	`}`
`11`	`11`	`module M {`
`12`		`- export class I { } // error`
	`12`	`+ export class I { }`
`13`	`13`	`}`
`14`	`14`
`15`	`15`	`module M {`
Original file line number	Diff line number	Diff line change
`@@ -3,7 +3,7 @@ module M {`
`3`	`3`	`export interface I { }`
`4`	`4`	`}`
`5`	`5`	`module M {`
`6`		`- export class I { } // error`
	`6`	`+ export class I { }`
`7`	`7`	`}`
`8`	`8`
`9`	`9`	`module M {`
`@@ -60,7 +60,7 @@ var M;`
`60`	`60`	`}`
`61`	`61`	`return I;`
`62`	`62`	`})();`
`63`		`- M.I = I; // error`
	`63`	`+ M.I = I;`
`64`	`64`	`})(M \|\| (M = {}));`
`65`	`65`	`var M;`
`66`	`66`	`(function (M) {`