Merge pull request microsoft#6869 from RyanCavanaugh/fix6811

RyanCavanaugh · RyanCavanaugh · commit a324176102b4 · 2016-02-04T15:48:03.000-08:00
Recognize `=` as equivalent to `?` in JSDoc signatures
diff --git a/src/compiler/parser.ts b/src/compiler/parser.ts
@@ -5743,6 +5743,9 @@ namespace ts {
             function parseJSDocParameter(): ParameterDeclaration {
                 const parameter = <ParameterDeclaration>createNode(SyntaxKind.Parameter);
                 parameter.type = parseJSDocType();
+                if (parseOptional(SyntaxKind.EqualsToken)) {
+                    parameter.questionToken = createNode(SyntaxKind.EqualsToken);
+                }
                 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
`@@ -5743,6 +5743,9 @@ namespace ts {`
`5743`	`5743`	`function parseJSDocParameter(): ParameterDeclaration {`
`5744`	`5744`	`const parameter = <ParameterDeclaration>createNode(SyntaxKind.Parameter);`
`5745`	`5745`	`parameter.type = parseJSDocType();`
	`5746`	`+ if (parseOptional(SyntaxKind.EqualsToken)) {`
	`5747`	`+ parameter.questionToken = createNode(SyntaxKind.EqualsToken);`
	`5748`	`+ }`
`5746`	`5749`	`return finishNode(parameter);`
`5747`	`5750`	`}`
`5748`	`5751`