Recognize = as equivalent to ? in JSDoc signatures

RyanCavanaugh · RyanCavanaugh · commit f7eac73a6f2e · 2016-02-03T00:09:38.000-08:00
Fixes microsoft#6811
diff --git a/src/compiler/parser.ts b/src/compiler/parser.ts
@@ -5727,6 +5727,9 @@ namespace ts {
             function parseJSDocParameter(): ParameterDeclaration {
                 const parameter = <ParameterDeclaration>createNode(SyntaxKind.Parameter);
                 parameter.type = parseJSDocType();
+                if (parseOptional(SyntaxKind.EqualsToken)) {
+                    parameter.questionToken = createNode(SyntaxKind.QuestionToken);
+                }
                 return finishNode(parameter);
             }
 
diff --git a/tests/cases/fourslash/jsDocFunctionSignatures2.ts b/tests/cases/fourslash/jsDocFunctionSignatures2.ts
@@ -0,0 +1,12 @@
+///<reference path="fourslash.ts" />
+
+// @allowNonTsExtensions: true
+// @Filename: Foo.js
+
+//// /** @type {function(string, boolean=): number} */
+//// var f6;
+//// 
+//// f6('', /**/false)
+
+goTo.marker();
+verify.currentSignatureHelpIs('f6(p0: string, p1?: boolean): number')

Original file line number	Diff line number	Diff line change
`@@ -5727,6 +5727,9 @@ namespace ts {`
`5727`	`5727`	`function parseJSDocParameter(): ParameterDeclaration {`
`5728`	`5728`	`const parameter = <ParameterDeclaration>createNode(SyntaxKind.Parameter);`
`5729`	`5729`	`parameter.type = parseJSDocType();`
	`5730`	`+ if (parseOptional(SyntaxKind.EqualsToken)) {`
	`5731`	`+ parameter.questionToken = createNode(SyntaxKind.QuestionToken);`
	`5732`	`+ }`
`5730`	`5733`	`return finishNode(parameter);`
`5731`	`5734`	`}`
`5732`	`5735`