Recognize = as equivalent to ? in JSDoc signatures

RyanCavanaugh · RyanCavanaugh · commit 8d0267c854e2 · 2016-02-05T10:47:28.000-08:00
Fixes microsoft#6811 (cherry picked from commit f7eac73)
diff --git a/src/compiler/parser.ts b/src/compiler/parser.ts
@@ -5758,6 +5758,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
`@@ -5758,6 +5758,9 @@ namespace ts {`
`5758`	`5758`	`function parseJSDocParameter(): ParameterDeclaration {`
`5759`	`5759`	`const parameter = <ParameterDeclaration>createNode(SyntaxKind.Parameter);`
`5760`	`5760`	`parameter.type = parseJSDocType();`
	`5761`	`+ if (parseOptional(SyntaxKind.EqualsToken)) {`
	`5762`	`+ parameter.questionToken = createNode(SyntaxKind.QuestionToken);`
	`5763`	`+ }`
`5761`	`5764`	`return finishNode(parameter);`
`5762`	`5765`	`}`
`5763`	`5766`