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