Fix handling of \r\n line terminators in Markdown

See valentjn/vscode-ltex#218.

CHANGELOG.md

@@ -14,6 +14,7 @@
 - Add support for Markdown tables as in [GitHub Flavored Markdown](https://github.github.com/gfm/#tables-extension-)
 - Enable `Add to dictionary` quick fix for Slovak rule IDs `MUZSKY_ROD_NEZIV_A`, `ZENSKY_ROD_A`, and `STREDNY_ROD_A` (fixes [vscode-ltex#221](https://github.com/valentjn/vscode-ltex/issues/221))
 - Remove superfluous spaces in messages of diagnostics
+- Fix handling of `\r\n` (Windows) line terminators in Markdown
 - Use Flexmark's YAML Front Matter extension to ignore YAML front matter in Markdown instead of own handling
 - Print Flexmark AST of Markdown documents to log when [`ltex.ltex-ls.logLevel`](https://valentjn.github.io/vscode-ltex/docs/settings.html#ltexltex-lsloglevel) is `"finest"`
 

ltexls-core/src/main/java/org/bsplines/ltexls/parsing/markdown/MarkdownAnnotatedTextBuilder.java

@@ -92,7 +92,8 @@ private void addMarkup(int newPos) {
 
     while (true) {
       if ((this.pos >= this.code.length()) || (this.pos >= newPos)) break;
-      int curPos = this.code.indexOf('\r', this.pos);
+      int curPos = this.code.indexOf("\r\n", this.pos);
+      if (curPos != -1) curPos += 1;
 
       if ((curPos == -1) || (curPos >= newPos)) {
         curPos = this.code.indexOf('\n', this.pos);

ltexls-core/src/test/java/org/bsplines/ltexls/parsing/markdown/MarkdownAnnotatedTextBuilderTest.java

@@ -42,6 +42,9 @@ public void test() throws IOException {
         + "Paragraph with\n"
         + "multiple lines and [link](example.com)\n",
         "Heading\nParagraph with multiple lines and link\n");
+    assertPlainText(
+        "This is a  \r\ntest.\r\n",
+        "This is a test.\n");
     assertPlainText(
         "# This is a © Test\n"
         + "Another [day – another](example.com) sentence\n",