Fix unintentional parentheses in optional method call emit (microsoft#440)

Author: rbuckton

internal/printer/printer.go

@@ -2268,7 +2268,7 @@ func (p *Printer) emitCallee(callee *ast.Expression, parentNode *ast.Node) {
 		// Parenthesize `new C` inside of a CallExpression so it is treated as `(new C)()` and not `new C()`
 		p.emitExpression(callee, ast.OperatorPrecedenceParentheses)
 	} else {
-		p.emitExpression(callee, ast.OperatorPrecedenceMember)
+		p.emitExpression(callee, core.IfElse(ast.IsOptionalChain(parentNode), ast.OperatorPrecedenceOptionalChain, ast.OperatorPrecedenceMember))
 	}
 }
 

internal/printer/printer_test.go

@@ -41,6 +41,7 @@ func TestEmit(t *testing.T) {
 		{title: "PropertyAccess#9", input: `0o1.b`, output: `0o1.b;`},
 		{title: "PropertyAccess#10", input: `10e1.b`, output: `10e1.b;`},
 		{title: "PropertyAccess#11", input: `10E1.b`, output: `10E1.b;`},
+		{title: "PropertyAccess#12", input: `a.b?.c`, output: `a.b?.c;`},
 		{title: "ElementAccess#1", input: `a[b]`, output: `a[b];`},
 		{title: "ElementAccess#2", input: `a?.[b]`, output: `a?.[b];`},
 		{title: "ElementAccess#3", input: `a?.[b].c`, output: `a?.[b].c;`},
@@ -56,6 +57,7 @@ func TestEmit(t *testing.T) {
 		{title: "CallExpression#10", input: `a?.<T>(b).c`, output: `a?.<T>(b).c;`},
 		{title: "CallExpression#11", input: `a<T, U>()`, output: `a<T, U>();`},
 		{title: "CallExpression#12", input: `a<T,>()`, output: `a<T,>();`},
+		{title: "CallExpression#13", input: `a?.b()`, output: `a?.b();`},
 		{title: "NewExpression#1", input: `new a`, output: `new a;`},
 		{title: "NewExpression#2", input: `new a.b`, output: `new a.b;`},
 		{title: "NewExpression#3", input: `new a()`, output: `new a();`},