Merge pull request #383 from commonmark/markdown-renderer-ordered-list-fix

MarkdownRenderer: Fix ordered list with long start number (fixes #382)

Author: robinst

CHANGELOG.md

@@ -15,6 +15,8 @@ with the exception that 0.x versions can break between minor versions.
 ### Fixed
 - `MarkdownRenderer`: Fix precedence for `nodeRendererFactory`: Factories passed
   to the builder can now override rendering for core node types.
+- `MarkdownRenderer`: Fix exception with ordered lists with a long first number
+  followed by a shorter one (#382)
 - Fix warning in Eclipse about "missing 'requires transitive'"
 - Fix Android incompatibility with `requireNonNullElseGet`
 

commonmark/src/main/java/org/commonmark/renderer/markdown/CoreMarkdownNodeRenderer.java

@@ -257,7 +257,7 @@ public void visit(ListItem listItem) {
             throw new IllegalStateException("Unknown list holder type: " + listHolder);
         }
         Integer contentIndent = listItem.getContentIndent();
-        String spaces = contentIndent != null ? repeat(" ", contentIndent - marker.length()) : " ";
+        String spaces = contentIndent != null ? repeat(" ", Math.max(contentIndent - marker.length(), 1)) : " ";
         writer.writePrefix(marker);
         writer.writePrefix(spaces);
         writer.pushPrefix(repeat(" ", marker.length() + spaces.length()));

commonmark/src/test/java/org/commonmark/renderer/markdown/MarkdownRendererTest.java

@@ -176,6 +176,13 @@ public void testOrderedListItemsFromAst() {
         assertRendering("", "2) Test\n", render(doc));
     }
 
+    @Test
+    public void testOrderedListItemsWithStartNumberLongerThanLaterNumber() {
+        var source = "10001.\n20.\n";
+        var doc = parse(source);
+        assertRendering(source, "10001. \n10002. \n", render(doc));
+    }
+
     // Inlines
 
     @Test