[PR #61] Fixes issue where LiteralInlineParser calls PostMatch while processor.Inline is type other than LiteralInline #802

New Issue

claunia · 2026-01-29T14:45:42Z

claunia commented

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

Original Pull Request: https://github.com/xoofx/markdig/pull/61

State: closed
Merged: Yes

I've found an odd situation where the AbbreviationParser throws a cast exception when processor.Inline is not of type LiteralInline. See lines 69-90 of LiteralInlineParser. In my case the type was EmphasisDelimiterInline and the current StringSlice was empty.

There are 2 ways I could have gone about this. I have chosen to assume that it is valid to expect the LiteralInlineParser to only call the PostMatch event when the processor.Inline type is LiteralInline so have changed the event raising code accordingly.

The alternative is that the consuming code should not make assumptions about the type of inline that the processor is handling and should handle it accordingly. Let me know if you'd rather I went down this route.

**Original Pull Request:** https://github.com/xoofx/markdig/pull/61 **State:** closed **Merged:** Yes --- I've found an odd situation where the `AbbreviationParser` throws a cast exception when `processor.Inline` is not of type `LiteralInline`. See lines 69-90 of `LiteralInlineParser`. In my case the type was `EmphasisDelimiterInline` and the current `StringSlice` was empty. There are 2 ways I could have gone about this. I have chosen to _assume_ that it is valid to expect the `LiteralInlineParser` to only call the `PostMatch` event when the `processor.Inline` type is `LiteralInline` so have changed the event raising code accordingly. The alternative is that the consuming code should not make assumptions about the type of inline that the processor is handling and should handle it accordingly. Let me know if you'd rather I went down this route.

claunia added the pull-request label 2026-01-29 14:45:42 +00:00

claunia closed this issue

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

claunia referenced this issue

2026-01-29 14:51:46 +00:00

[PR #802] [CLOSED] Add ConfigureHtmlRenderer to MarkdownPipelineBuilder #1228

Sign in to join this conversation.