Merge pull request microsoft#4991 from MartyIX/patch-1

DanielRosenwasser · DanielRosenwasser · commit 7d257b2a843d · 2015-09-28T10:34:08.000-07:00
Do not auto-generate names for anonymous class expressions
diff --git a/src/compiler/emitter.ts b/src/compiler/emitter.ts
@@ -4529,8 +4529,10 @@ var __awaiter = (this && this.__awaiter) || function (thisArg, _arguments, Promi
 
                 write("class");
 
-                // check if this is an "export default class" as it may not have a name. Do not emit the name if the class is decorated.
-                if ((node.name || !(node.flags & NodeFlags.Default)) && !thisNodeIsDecorated) {
+                // emit name if
+                // - node has a name
+                // - this is default export and target is not ES6 (for ES6 `export default` does not need to be compiled downlevel)                
+                if ((node.name || (node.flags & NodeFlags.Default && languageVersion < ScriptTarget.ES6)) && !thisNodeIsDecorated) {
                     write(" ");
                     emitDeclarationName(node);
                 }
diff --git a/tests/baselines/reference/classExpressionES63.js b/tests/baselines/reference/classExpressionES63.js
@@ -7,7 +7,7 @@ c.c;
 
 
 //// [classExpressionES63.js]
-let C = class class_1 extends class class_2 extends class class_3 {
+let C = class extends class extends class {
     constructor() {
         this.a = 1;
     }

Original file line number	Diff line number	Diff line change
`@@ -7,7 +7,7 @@ c.c;`
`7`	`7`
`8`	`8`
`9`	`9`	`//// [classExpressionES63.js]`
`10`		`-let C = class class_1 extends class class_2 extends class class_3 {`
	`10`	`+let C = class extends class extends class {`
`11`	`11`	`constructor() {`
`12`	`12`	`this.a = 1;`
`13`	`13`	`}`