Explicit handling for more nodes in reduceEachChild/visitEachChild

Author: rbuckton

src/compiler/factory.ts

@@ -1441,6 +1441,7 @@ namespace ts {
             // checked is to not pass the file to getEmitResolver.
             const emitResolver = getDiagnosticsProducingTypeChecker().getEmitResolver((options.outFile || options.out) ? undefined : sourceFile);
 
+            performance.emit("beforeEmit");
             const emitStart = performance.mark();
 
             const emitResult = emitFiles(
@@ -1449,6 +1450,7 @@ namespace ts {
                 sourceFile);
 
             performance.measure("emitTime", emitStart);
+            performance.emit("afterEmit");
 
             return emitResult;
         }

src/compiler/program.ts

@@ -351,7 +351,9 @@ namespace ts {
             value = ensureIdentifier(value, /*reuseIdentifierExpressions*/ true, location, emitTempVariableAssignment);
             return createConditional(
                 createStrictEquality(value, createVoidZero()),
+                createToken(SyntaxKind.QuestionToken),
                 defaultValue,
+                createToken(SyntaxKind.ColonToken),
                 value
             );
         }

src/compiler/transformers/destructuring.ts

@@ -734,8 +734,10 @@ namespace ts {
                 statements.push(
                     setOriginalNode(
                         createClassDeclaration(
+                            /*decorators*/ undefined,
                             /*modifiers*/ undefined,
                             name,
+                            /*typeParameters*/ undefined,
                             node.heritageClauses,
                             node.members,
                             /*location*/ node
@@ -908,7 +910,7 @@ namespace ts {
                     setNodeEmitFlags(node, NodeEmitFlags.NoSubstitution);
                     let transformedUnaryExpression: BinaryExpression;
                     if (node.kind === SyntaxKind.PostfixUnaryExpression) {
-                        transformedUnaryExpression = createBinaryWithOperatorToken(
+                        transformedUnaryExpression = createBinary(
                             operand,
                             createNode(operator === SyntaxKind.PlusPlusToken ? SyntaxKind.PlusEqualsToken : SyntaxKind.MinusEqualsToken),
                             createLiteral(1),

src/compiler/transformers/module/module.ts

@@ -720,7 +720,9 @@ namespace ts {
                     createAssignment(
                         name,
                         createClassExpression(
+                            /*modifiers*/ undefined,
                             node.name,
+                            /*typeParameters*/ undefined,
                             node.heritageClauses,
                             node.members,
                             /*location*/ node

src/compiler/transformers/module/system.ts

@@ -506,8 +506,10 @@ namespace ts {
                 //      ${members}
                 //  }
                 const classDeclaration = createClassDeclaration(
+                    /*decorators*/ undefined,
                     visitNodes(node.modifiers, visitor, isModifier),
                     name,
+                    /*typeParameters*/ undefined,
                     visitNodes(node.heritageClauses, visitor, isHeritageClause),
                     transformClassMembers(node, hasExtendsClause),
                     /*location*/ node
@@ -548,7 +550,7 @@ namespace ts {
             }
             else if (isDecoratedClass) {
                 if (isDefaultExternalModuleExport(node)) {
-                    statements.push(createExportDefault(getLocalName(node)));
+                    statements.push(createExportAssignment(/*isExportEquals*/ false, getLocalName(node)));
                 }
                 else if (isNamedExternalModuleExport(node)) {
                     statements.push(createExternalModuleExport(name));
@@ -660,7 +662,9 @@ namespace ts {
             //  }
             const classExpression: Expression = setOriginalNode(
                 createClassExpression(
+                    /*modifiers*/ undefined,
                     name,
+                    /*typeParameters*/ undefined,
                     visitNodes(node.heritageClauses, visitor, isHeritageClause),
                     transformClassMembers(node, hasExtendsClause),
                     /*location*/ location
@@ -743,7 +747,9 @@ namespace ts {
 
             const classExpression = setOriginalNode(
                 createClassExpression(
+                    /*modifiers*/ undefined,
                     node.name,
+                    /*typeParameters*/ undefined,
                     heritageClauses,
                     members,
                     /*location*/ node
@@ -1871,7 +1877,9 @@ namespace ts {
                     createTypeOf(createIdentifier("Symbol")),
                     createLiteral("function")
                 ),
+                createToken(SyntaxKind.QuestionToken),
                 createIdentifier("Symbol"),
+                createToken(SyntaxKind.ColonToken),
                 createIdentifier("Object")
             );
         }
@@ -1957,6 +1965,7 @@ namespace ts {
         function visitExpressionWithTypeArguments(node: ExpressionWithTypeArguments): ExpressionWithTypeArguments {
             const expression = visitNode(node.expression, visitor, isLeftHandSideExpression);
             return createExpressionWithTypeArguments(
+                /*typeArguments*/ undefined,
                 expression,
                 node
             );
@@ -2360,6 +2369,7 @@ namespace ts {
         function visitAwaitExpression(node: AwaitExpression): Expression {
             return setOriginalNode(
                 createYield(
+                    /*asteriskToken*/ undefined,
                     visitNode(node.expression, visitor, isExpression),
                     /*location*/ node
                 ),