Completely replace JSDocNonNullableType parsing with OptionalType (#96)

Author: sandersn

internal/compiler/parser.go

@@ -2233,17 +2233,13 @@ func (p *Parser) parsePostfixTypeOrHigher() *ast.Node {
 	typeNode := p.parseNonArrayType()
 	for !p.hasPrecedingLineBreak() {
 		switch p.token {
-		case ast.KindExclamationToken:
-			p.nextToken()
-			typeNode = p.factory.NewJSDocNonNullableType(typeNode)
-			p.finishNode(typeNode, pos)
 		case ast.KindQuestionToken:
-			// If next token is start of a type we have a conditional type
+			// If next token is start of a type we are in the middle of a conditional type
 			if p.lookAhead(p.nextIsStartOfType) {
 				return typeNode
 			}
 			p.nextToken()
-			typeNode = p.factory.NewJSDocNullableType(typeNode)
+			typeNode = p.factory.NewOptionalTypeNode(typeNode)
 			p.finishNode(typeNode, pos)
 		case ast.KindOpenBracketToken:
 			p.parseExpected(ast.KindOpenBracketToken)
@@ -3155,17 +3151,7 @@ func (p *Parser) parseTupleElementType() *ast.TypeNode {
 		p.finishNode(result, pos)
 		return result
 	}
-	typeNode := p.parseType()
-	if typeNode.Kind == ast.KindJSDocNullableType {
-		nullableType := typeNode.AsJSDocNullableType()
-		if typeNode.Loc.Pos() == nullableType.TypeNode.Loc.Pos() {
-			result := p.factory.NewOptionalTypeNode(nullableType.TypeNode)
-			result.Loc = typeNode.Loc
-			result.Flags = typeNode.Flags
-			return result
-		}
-	}
-	return typeNode
+	return p.parseType()
 }
 
 func (p *Parser) parseParenthesizedType() *ast.Node {