InvalidCastException in SmartyPantsInlineParser #39

New Issue

claunia · 2026-01-29T14:22:18Z

claunia commented

2026-01-29 14:22:18 +00:00

Originally created by @Jither on GitHub (Jun 28, 2016).

No pull request for this one, sorry ---(*) until Github figures out I'm human. 😈 Besides, I'm not quite sure I follow the reasoning in the SmartyPants parser (I know changes were made to fix the table rendering).

However, test case:

        [Test]
        public void TestInvalidCastBug()
        {
            var pipeline = new MarkdownPipelineBuilder()
                .UseSmartyPants()
                .Build();
            var md = "\"Curly\" quotations and dashes --- like that one.";
            Assert.DoesNotThrow(() => Markdown.Parse(md, pipeline));
        }

Markdig throws an InvalidCastException:
Unable to cast object of type 'System.String' to type 'System.Collections.Generic.List`1[Markdig.Extensions.SmartyPants.SmartyPant]

Seems to be due to the dashes logic writing String.Empty, overwriting the list created by the quotation marks, in ParserState[Index].

(*) Written too many m-dashes today, it seems. ;-)

Originally created by @Jither on GitHub (Jun 28, 2016). No pull request for this one, sorry ---(*) until Github figures out I'm human. :smiling_imp: Besides, I'm not quite sure I follow the reasoning in the SmartyPants parser (I know changes were made to fix the table rendering). However, test case: ``` csharp [Test] public void TestInvalidCastBug() { var pipeline = new MarkdownPipelineBuilder() .UseSmartyPants() .Build(); var md = "\"Curly\" quotations and dashes --- like that one."; Assert.DoesNotThrow(() => Markdown.Parse(md, pipeline)); } ``` Markdig throws an InvalidCastException: Unable to cast object of type 'System.String' to type 'System.Collections.Generic.List`1[Markdig.Extensions.SmartyPants.SmartyPant] Seems to be due to the dashes logic writing `String.Empty`, overwriting the list created by the quotation marks, in `ParserState[Index]`. (*) Written too many m-dashes today, it seems. ;-)

claunia referenced this issue

2026-01-29 14:45:23 +00:00

[PR #39] [MERGED] Bugfix/cell after colspan #776

claunia referenced this issue

2026-01-29 14:45:26 +00:00

[PR #39] Bugfix/cell after colspan #781

Sign in to join this conversation.