A couple of small simplifications

ahejlsberg · ahejlsberg · commit ceb78f960783 · 2017-05-01T09:10:51.000-07:00
diff --git a/src/compiler/checker.ts b/src/compiler/checker.ts
@@ -3051,7 +3051,7 @@ namespace ts {
                         buildSymbolDisplay(parent, writer, enclosingDeclaration, SymbolFlags.Type, SymbolFormatFlags.None, nextFlags);
                         // In a literal enum type with a single member E { A }, E and E.A denote the
                         // same type. We always display this type simply as E.
-                        if (getDeclaredTypeOfSymbol(parent).flags & TypeFlags.Union) {
+                        if (getDeclaredTypeOfSymbol(parent) !== type) {
                             writePunctuation(writer, SyntaxKind.DotToken);
                             appendSymbolNameOnly(type.symbol, writer);
                         }
@@ -4958,9 +4958,7 @@ namespace ts {
                         for (const member of (<EnumDeclaration>declaration).members) {
                             const memberType = getLiteralType(getEnumMemberValue(member), enumCount, getSymbolOfNode(member));
                             getSymbolLinks(getSymbolOfNode(member)).declaredType = memberType;
-                            if (!contains(memberTypeList, memberType)) {
-                                memberTypeList.push(memberType);
-                            }
+                            memberTypeList.push(memberType);
                         }
                     }
                 }

Original file line number	Diff line number	Diff line change
`@@ -3051,7 +3051,7 @@ namespace ts {`
`3051`	`3051`	`buildSymbolDisplay(parent, writer, enclosingDeclaration, SymbolFlags.Type, SymbolFormatFlags.None, nextFlags);`
`3052`	`3052`	`// In a literal enum type with a single member E { A }, E and E.A denote the`
`3053`	`3053`	`// same type. We always display this type simply as E.`
`3054`		`- if (getDeclaredTypeOfSymbol(parent).flags & TypeFlags.Union) {`
	`3054`	`+ if (getDeclaredTypeOfSymbol(parent) !== type) {`
`3055`	`3055`	`writePunctuation(writer, SyntaxKind.DotToken);`
`3056`	`3056`	`appendSymbolNameOnly(type.symbol, writer);`
`3057`	`3057`	`}`
`@@ -4958,9 +4958,7 @@ namespace ts {`
`4958`	`4958`	`for (const member of (<EnumDeclaration>declaration).members) {`
`4959`	`4959`	`const memberType = getLiteralType(getEnumMemberValue(member), enumCount, getSymbolOfNode(member));`
`4960`	`4960`	`getSymbolLinks(getSymbolOfNode(member)).declaredType = memberType;`
`4961`		`- if (!contains(memberTypeList, memberType)) {`
`4962`		`- memberTypeList.push(memberType);`
`4963`		`- }`
	`4961`	`+ memberTypeList.push(memberType);`
`4964`	`4962`	`}`
`4965`	`4963`	`}`
`4966`	`4964`	`}`