Emit trailing comments

Author: sheetalkamat

src/compiler/emitter.ts

@@ -1930,6 +1930,7 @@ module ts {
 
             function emitTrailingDeclarationComments(node: Declaration) {
                 var trailingComments = getTrailingComments(currentSourceFile.text, node.end);
+                emitComments(trailingComments, writer, writeComment);
             }
 
             if (compilerOptions.sourceMap) {

tests/baselines/reference/AmbientModuleAndAmbientWithSameNameAndCommonRoot.js

@@ -28,4 +28,4 @@ var p = new A.Point(0, 0); // unexpected error here, bug 840000
 //// [test.js]
 var p;
 var p = A.Point.Origin;
-var p = new A.Point(0, 0);
+var p = new A.Point(0, 0);// unexpected error here, bug 840000 

tests/baselines/reference/AmbientModuleAndNonAmbientClassWithSameNameAndCommonRoot.js

@@ -37,4 +37,4 @@ var A;
 //// [test.js]
 var p;
 var p = A.Point.Origin;
-var p = new A.Point(0, 0);
+var p = new A.Point(0, 0);// unexpected error here, bug 840000 

tests/baselines/reference/ClassAndModuleThatMergeWithStaticFunctionAndExportedFunctionThatShareAName.js

@@ -30,15 +30,15 @@ var Point = (function () {
     }
     Point.Origin = function () {
         return { x: 0, y: 0 };
-    };
+    };// unexpected error here bug 840246 
     return Point;
 })();
 var Point;
 (function (Point) {
     function Origin() {
         return null;
     }
-    Point.Origin = Origin;
+    Point.Origin = Origin;//expected duplicate identifier error 
 })(Point || (Point = {}));
 var A;
 (function (A) {
@@ -49,15 +49,15 @@ var A;
         }
         Point.Origin = function () {
             return { x: 0, y: 0 };
-        };
+        };// unexpected error here bug 840246 
         return Point;
     })();
     A.Point = Point;
     (function (Point) {
         function Origin() {
             return "";
         }
-        Point.Origin = Origin;
+        Point.Origin = Origin;//expected duplicate identifier error 
     })(A.Point || (A.Point = {}));
     var Point = A.Point;
 })(A || (A = {}));

tests/baselines/reference/ClassAndModuleThatMergeWithStaticFunctionAndNonExportedFunctionThatShareAName.js

@@ -37,7 +37,7 @@ var Point;
 (function (Point) {
     function Origin() {
         return "";
-    }
+    }// not an error, since not exported 
 })(Point || (Point = {}));
 var A;
 (function (A) {
@@ -55,7 +55,7 @@ var A;
     (function (Point) {
         function Origin() {
             return "";
-        }
+        }// not an error since not exported 
     })(A.Point || (A.Point = {}));
     var Point = A.Point;
 })(A || (A = {}));

tests/baselines/reference/ClassAndModuleThatMergeWithStaticVariableAndExportedVarThatShareAName.js

@@ -33,7 +33,7 @@ var Point = (function () {
 })();
 var Point;
 (function (Point) {
-    Point.Origin = "";
+    Point.Origin = "";//expected duplicate identifier error 
 })(Point || (Point = {}));
 var A;
 (function (A) {
@@ -47,7 +47,7 @@ var A;
     })();
     A.Point = Point;
     (function (Point) {
-        Point.Origin = "";
+        Point.Origin = "";//expected duplicate identifier error 
     })(A.Point || (A.Point = {}));
     var Point = A.Point;
 })(A || (A = {}));

tests/baselines/reference/ClassAndModuleThatMergeWithStaticVariableAndNonExportedVarThatShareAName.js

@@ -33,7 +33,7 @@ var Point = (function () {
 })();
 var Point;
 (function (Point) {
-    var Origin = "";
+    var Origin = "";// not an error, since not exported 
 })(Point || (Point = {}));
 var A;
 (function (A) {
@@ -47,7 +47,7 @@ var A;
     })();
     A.Point = Point;
     (function (Point) {
-        var Origin = "";
+        var Origin = "";// not an error since not exported 
     })(A.Point || (A.Point = {}));
     var Point = A.Point;
 })(A || (A = {}));

tests/baselines/reference/ClassAndModuleWithSameNameAndCommonRoot.js

@@ -69,7 +69,7 @@ var X;
 //// [test.js]
 //var cl: { x: number; y: number; }
 var cl = new X.Y.Point(1, 1);
-var cl = X.Y.Point.Origin;
+var cl = X.Y.Point.Origin;// error not expected here same as bug 83996 ? 
 //// [simple.js]
 var A = (function () {
     function A() {

tests/baselines/reference/FunctionAndModuleWithSameNameAndCommonRoot.js

@@ -64,7 +64,7 @@ var fn;
 var fn = A.Point;
 var cl;
 var cl = A.Point();
-var cl = A.Point.Origin;
+var cl = A.Point.Origin;// not expected to be an error. 
 //// [simple.js]
 var B;
 (function (B) {
@@ -78,7 +78,7 @@ var B;
     var Point = B.Point;
 })(B || (B = {}));
 var fn;
-var fn = B.Point;
+var fn = B.Point;// not expected to be an error. bug 840000: [corelang] Function of fundule not assignalbe as expected 
 var cl;
 var cl = B.Point();
 var cl = B.Point.Origin;

tests/baselines/reference/InvalidNonInstantiatedModule.js

@@ -9,5 +9,5 @@ var x: typeof M; // Error only a namespace
 
 
 //// [InvalidNonInstantiatedModule.js]
-var m = M;
-var x;
+var m = M;// Error, not instantiated can not be used as var 
+var x;// Error only a namespace 

tests/baselines/reference/ModuleWithExportedAndNonExportedFunctions.js

@@ -51,7 +51,7 @@ var A;
 var fn;
 var fn = A.fn;
 var fng;
-var fng = A.fng;
+var fng = A.fng;// bug 838015 
 // these should be errors since the functions are not exported
 var fn2 = A.fn2;
 var fng2 = A.fng2;

tests/baselines/reference/ParameterList7.js

@@ -9,6 +9,6 @@ class C1 {
 var C1 = (function () {
     function C1(p3) {
         this.p3 = p3;
-    }
+    }// OK 
     return C1;
 })();

tests/baselines/reference/additionOperatorWithInvalidOperands.js

@@ -70,7 +70,7 @@ var r2 = a + b;
 var r3 = a + c;
 // number + every type except any and string
 var r4 = b + a;
-var r5 = b + b;
+var r5 = b + b;// number + number is valid 
 var r6 = b + c;
 // object + every type except any and string
 var r7 = c + a;

tests/baselines/reference/additionOperatorWithTypeParameter.js

@@ -54,18 +54,18 @@ function foo(t, u) {
     var g;
     var f;
     // type parameter as left operand
-    var r1 = t + a;
+    var r1 = t + a;// ok, one operand is any 
     var r2 = t + b;
     var r3 = t + c;
-    var r4 = t + d;
+    var r4 = t + d;// ok, one operand is string 
     var r5 = t + e;
     var r6 = t + g;
     var r7 = t + f;
     // type parameter as right operand
-    var r8 = a + t;
+    var r8 = a + t;// ok, one operand is any 
     var r9 = b + t;
     var r10 = c + t;
-    var r11 = d + t;
+    var r11 = d + t;// ok, one operand is string 
     var r12 = e + t;
     var r13 = g + t;
     var r14 = f + t;

tests/baselines/reference/aliasBug.js

@@ -46,7 +46,7 @@ var provide = foo;
 var booz = foo.bar.baz;
 var p = new provide.Provide();
 function use() {
-    var p1;
+    var p1;// error here, but should be okay 
     var p2;
     var p3;
     var p22 = new provide.Provide();

tests/baselines/reference/aliasOnMergedModuleInterface.js

@@ -25,4 +25,4 @@ var x: foo.A = foo.bar("hello"); // foo.A should be ok but foo.bar should be err
 //// [aliasOnMergedModuleInterface_1.js]
 var z;
 z.bar("hello");
-var x = foo.bar("hello");
+var x = foo.bar("hello");// foo.A should be ok but foo.bar should be error 

tests/baselines/reference/ambiguousOverloadResolution.js

@@ -28,4 +28,4 @@ var B = (function (_super) {
     return B;
 })(A);
 var x;
-var t = f(x, x);
+var t = f(x, x);// Not an error 

tests/baselines/reference/anyAsConstructor.js

@@ -18,4 +18,4 @@ var a = new x();
 var b = new x('hello');
 var c = new x(x);
 // grammar allows this for constructors
-var d = new x(x);
+var d = new x(x);// no error 

tests/baselines/reference/anyAssignabilityInInheritance.js

@@ -90,40 +90,40 @@ var r3 = foo3(a); // any
 
 //// [anyAssignabilityInInheritance.js]
 var a;
-var r3 = foo2(a);
-var r3 = foo3(a);
-var r3 = foo3(a);
-var r3 = foo3(a);
-var r3 = foo3(a);
-var r3 = foo3(a);
-var r3 = foo3(a);
-var r3 = foo3(a);
+var r3 = foo2(a);// any, not a subtype of number so it skips that overload, is a subtype of itself so it picks second (if truly ambiguous it would pick first overload) 
+var r3 = foo3(a);// any 
+var r3 = foo3(a);// any 
+var r3 = foo3(a);// any 
+var r3 = foo3(a);// any 
+var r3 = foo3(a);// any 
+var r3 = foo3(a);// any 
+var r3 = foo3(a);// any 
 var A = (function () {
     function A() {
     }
     return A;
 })();
-var r3 = foo3(a);
+var r3 = foo3(a);// any 
 var A2 = (function () {
     function A2() {
     }
     return A2;
 })();
-var r3 = foo3(a);
-var r3 = foo3(a);
-var r3 = foo3(a);
+var r3 = foo3(a);// any 
+var r3 = foo3(a);// any 
+var r3 = foo3(a);// any 
 var E;
 (function (E) {
     E[E["A"] = 0] = "A";
 })(E || (E = {}));
-var r3 = foo3(a);
+var r3 = foo3(a);// any 
 function f() {
 }
 var f;
 (function (f) {
     f.bar = 1;
 })(f || (f = {}));
-var r3 = foo3(a);
+var r3 = foo3(a);// any 
 var CC = (function () {
     function CC() {
     }
@@ -133,6 +133,6 @@ var CC;
 (function (CC) {
     CC.bar = 1;
 })(CC || (CC = {}));
-var r3 = foo3(a);
-var r3 = foo3(a);
-var r3 = foo3(a);
+var r3 = foo3(a);// any 
+var r3 = foo3(a);// any 
+var r3 = foo3(a);// any