Adopt Strada-style object type printing. (microsoft#433)

Author: DanielRosenwasser

internal/checker/printer.go

@@ -364,27 +364,20 @@ func (p *Printer) printAnonymousType(t *Type) {
 		}
 	}
 	p.print("{")
-	var tail bool
+	hasMembers := false
 	for _, sig := range callSignatures {
-		if tail {
-			p.print(",")
-		}
 		p.print(" ")
 		p.printSignature(sig, ": ")
-		tail = true
+		p.print(";")
+		hasMembers = true
 	}
 	for _, sig := range constructSignatures {
-		if tail {
-			p.print(",")
-		}
 		p.print(" new")
 		p.printSignature(sig, ": ")
-		tail = true
+		p.print(";")
+		hasMembers = true
 	}
 	for _, info := range p.c.getIndexInfosOfType(t) {
-		if tail {
-			p.print(",")
-		}
 		if info.isReadonly {
 			p.print(" readonly")
 		}
@@ -394,12 +387,10 @@ func (p *Printer) printAnonymousType(t *Type) {
 		p.printType(info.keyType)
 		p.print("]: ")
 		p.printType(info.valueType)
-		tail = true
+		p.print(";")
+		hasMembers = true
 	}
 	for _, prop := range props {
-		if tail {
-			p.print(",")
-		}
 		if p.c.isReadonlySymbol(prop) {
 			p.print(" readonly")
 		}
@@ -410,9 +401,10 @@ func (p *Printer) printAnonymousType(t *Type) {
 		}
 		p.print(": ")
 		p.printType(p.c.getTypeOfSymbol(prop))
-		tail = true
+		p.print(";")
+		hasMembers = true
 	}
-	if tail {
+	if hasMembers {
 		p.print(" ")
 	}
 	p.print("}")

testdata/baselines/reference/compiler/allowSyntheticDefaultImports9.types

@@ -9,18 +9,18 @@ export function bar();
 
 === a.ts ===
 import { default as Foo } from "./b";
->default : { foo: () => any, bar: () => any }
->Foo : { foo: () => any, bar: () => any }
+>default : { foo: () => any; bar: () => any; }
+>Foo : { foo: () => any; bar: () => any; }
 
 Foo.bar();
 >Foo.bar() : any
 >Foo.bar : () => any
->Foo : { foo: () => any, bar: () => any }
+>Foo : { foo: () => any; bar: () => any; }
 >bar : () => any
 
 Foo.foo();
 >Foo.foo() : any
 >Foo.foo : () => any
->Foo : { foo: () => any, bar: () => any }
+>Foo : { foo: () => any; bar: () => any; }
 >foo : () => any
 

testdata/baselines/reference/compiler/symbolLinkDeclarationEmitModuleNamesRootDir.types

@@ -12,13 +12,13 @@ import { ControllerClass } from './application';
 >ControllerClass : error
 
 import { BindingKey } from '@loopback/context';
->BindingKey : { new<T>(): BindingKey<T>, create: <T>(ctor: T) => BindingKey<T>, prototype: BindingKey<any> }
+>BindingKey : { new<T>(): BindingKey<T>; create: <T>(ctor: T) => BindingKey<T>; prototype: BindingKey<any>; }
 
 export const CONTROLLER_CLASS = BindingKey.create<ControllerClass>(null as any); // line in question
 >CONTROLLER_CLASS : BindingKey<ControllerClass>
 >BindingKey.create<ControllerClass>(null as any) : BindingKey<ControllerClass>
 >BindingKey.create : <T>(ctor: T) => BindingKey<T>
->BindingKey : { new<T>(): BindingKey<T>, create: <T>(ctor: T) => BindingKey<T>, prototype: BindingKey<any> }
+>BindingKey : { new<T>(): BindingKey<T>; create: <T>(ctor: T) => BindingKey<T>; prototype: BindingKey<any>; }
 >create : <T>(ctor: T) => BindingKey<T>
 >null as any : any