Typo in function emitExternalImportDeclaration?

[kzc]

const isNakedImport = SyntaxKind.ImportDeclaration && !(<ImportDeclaration>node).importClause;

https://github.com/Microsoft/TypeScript/blob/ddb5a00410d695d35b55c5482876a16f7c696c4b/src/compiler/emitter.ts#L6590

Possibly missing node.kind === before SyntaxKind.ImportDeclaration ?

Noticed it when running uglifyjs lib/tsc.js -c

WARN: Condition left of && always true [lib/tsc.js:34409,48]

 34409                              var isNakedImport = 230 && !node.importClause;

[DanielRosenwasser]

Nice catch @kzc! We'd be willing to take a PR for the fix with a test where this surfaces.

[kzc]

Just thought I'd report the obvious issue. You can take over.

[kzc]

A previous patch confirms the suggested fix:

956d7a8#diff-20a9fa4d3e9d09f60c275a3dda245867L5008

Surprised by its author - coding royalty!

[RyanCavanaugh]

If you could avoid bugs by being smart, we wouldn't need TypeScript 😉