Fixed jsDoc parser - no longer omits asterisks in the middle (if the …

…line does not start with asterisk) and additional case for whitespaces being ignored
microsoft · Dec 12, 2016 · aa4a0b6 · aa4a0b6
1 parent 1907064
commit aa4a0b6
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 2 deletions.
diff --git a/src/compiler/parser.ts b/src/compiler/parser.ts
@@ -6337,7 +6337,7 @@ namespace ts {
                                 break;
                             case SyntaxKind.AsteriskToken:
                                 const asterisk = scanner.getTokenText();
-                                if (state === JSDocState.SawAsterisk) {
+                                if (state === JSDocState.SawAsterisk || state === JSDocState.SavingComments) {
                                     // If we've already seen an asterisk, then we can no longer parse a tag on this line
                                     state = JSDocState.SavingComments;
                                     pushComment(asterisk);
@@ -6369,6 +6369,8 @@ namespace ts {
                             case SyntaxKind.EndOfFileToken:
                                 break;
                             default:
+                                // anything other than whitespace or asterisk at the beginning of the line starts the comment text
+                                state = JSDocState.SavingComments;
                                 pushComment(scanner.getTokenText());
                                 break;
                         }

diff --git a/tests/cases/fourslash/commentsLinePreservation.ts b/tests/cases/fourslash/commentsLinePreservation.ts
@@ -108,6 +108,7 @@
 ////     /** 
 ////       * This is firstLine
 //// This is second Line
+//// [1]: third * line
 //// @param param1 first Line text
 //// second line text
 //// */
@@ -145,6 +146,6 @@ verify.quickInfos({
     l: ["function l(param1: string): void", "This is firstLine\nThis is second Line"],
     9: ["(parameter) param1: string", "first Line text\nblank line that shouldnt be shown when starting this \nsecond time information about the param again"],
 
-    m: ["function m(param1: string): void", "This is firstLine\nThis is second Line"],
+    m: ["function m(param1: string): void", "This is firstLine\nThis is second Line\n[1]: third * line"],
     10: ["(parameter) param1: string", "first Line text\nsecond line text"]
 });