Parse docs with line endings: LF, CR, CRLF

Author: danon

ReadMe.md

@@ -239,9 +239,10 @@ The original implementation is in <https://github.com/jasny/phpdoc-parser>.
 
 Features fixed and improved so far:
 
+- Tags `@param` failed to handle multiline tags properly, now it's fixed
 - Typo `"summery"` to `"summary"`
 - Improper parsing summary in multiple lines
 - Added support for `@version` tag
 - `@method` interprets `static` properly
-- Tags `@param` failed to handle multiline tags properly
 - Fixed improper parsing of unclosed or improperly closed doc block
+- Previous parser failed parsing for line endings `CR` and `CRLF`, now it supports: `LF`, `CR`, `CRLF`.

src/PhpdocParser.php

@@ -41,7 +41,7 @@ private function summaryLinesAndTagList(string $docBlock): array
 
     private function tagsAndLines(string $docBlock): array
     {
-        $regex = Pattern::of('^
+        $regex = Pattern::of('(*ANYCRLF)^
               \s*
               \*?
               (?:

test/Feature/Test.php

@@ -64,6 +64,42 @@ public function shouldParseStandardExample()
         ]);
     }
 
+    /**
+     * @test
+     */
+    public function shouldParseCrlf()
+    {
+        // given
+        $crlfDoc = "/**\r\n * @deprecated\r\n */";
+        // when, then
+        $this->assertParses($crlfDoc, ['deprecated' => true,]);
+    }
+
+    /**
+     * @test
+     */
+    public function shouldParseCr()
+    {
+        // given
+        $crlfDoc = "/**\r * @deprecated\r */";
+        // when, then
+        $this->assertParses($crlfDoc, ['deprecated' => true,]);
+    }
+
+    /**
+     * @test
+     */
+    public function shouldParseCrNewlines()
+    {
+        // given
+        $crlfDoc = "/** summary\r*\r*description */";
+        // when, then
+        $this->assertParses($crlfDoc, [
+            'summary'     => 'summary',
+            'description' => 'description',
+        ]);
+    }
+
     /**
      * @test
      * @dataProvider improperDocBlocks