Apply foreach on class StringLineGroup got phantom empty line, is it expected? #645

New Issue

claunia · 2026-01-29T14:41:56Z

claunia commented

2026-01-29 14:41:56 +00:00

Originally created by @Akarinnnnn on GitHub (Dec 11, 2023).

I' m writing code to dump a FencedCodeBlock, here is the function:

        private static void HandleCodeBlock(FencedCodeBlock codeblock, int level)
        {
            Console.ForegroundColor = ConsoleColor.DarkYellow;
            if (!string.IsNullOrEmpty(codeblock.Info))
            {
                PrintIndent(level);
                Console.WriteLine($"lang: {codeblock.Info}, info args: {codeblock.Arguments}");
            }
            Console.ForegroundColor = ConsoleColor.White;

            foreach (var line in codeblock.Lines)
                Console.WriteLine(line);
        }

Then I see output contains suspicious empty line like this:

  FencedCodeBlock: 36/692-735 @1
 characters.sifeng.voice("c1s1", 3)

And the corresponding markdown is:
```
characters.sifeng.voice("c1s1", 3)
```

After some investigation, I found the enumerator of StringLineGroup returns internal array's enumerator directly. The array contains reserved slot for future item adding, that's the reason why we see phantom empty lines.

Shall we change the GetEnumerator() for StringLineGroup? I can provide some help.

Originally created by @Akarinnnnn on GitHub (Dec 11, 2023). I' m writing code to dump a `FencedCodeBlock`, here is the function: ```csharp private static void HandleCodeBlock(FencedCodeBlock codeblock, int level) { Console.ForegroundColor = ConsoleColor.DarkYellow; if (!string.IsNullOrEmpty(codeblock.Info)) { PrintIndent(level); Console.WriteLine($"lang: {codeblock.Info}, info args: {codeblock.Arguments}"); } Console.ForegroundColor = ConsoleColor.White; foreach (var line in codeblock.Lines) Console.WriteLine(line); } ``` Then I see output contains suspicious empty line like this: ```plaintext FencedCodeBlock: 36/692-735 @1 characters.sifeng.voice("c1s1", 3) ``` And the corresponding markdown is: \`\`\` characters.sifeng.voice("c1s1", 3) \`\`\` After some investigation, I found the enumerator of `StringLineGroup` returns internal array's enumerator directly. The array contains reserved slot for future item adding, that's the reason why we see phantom empty lines. Shall we change the `GetEnumerator()` for `StringLineGroup`? I can provide some help.

claunia added the bug PR Welcome! labels 2026-01-29 14:41:56 +00:00

claunia closed this issue

2026-01-29 14:41:56 +00:00

claunia commented

2026-01-29 14:41:57 +00:00

@xoofx commented on GitHub (Dec 11, 2023):

Shall we change the GetEnumerator() for StringLineGroup? I can provide some help.

Yep most likely, you can make a PR for a fix, thanks!

@xoofx commented on GitHub (Dec 11, 2023): > Shall we change the `GetEnumerator()` for `StringLineGroup`? I can provide some help. Yep most likely, you can make a PR for a fix, thanks!

claunia commented

2026-01-29 14:41:58 +00:00

@Akarinnnnn commented on GitHub (Dec 11, 2023):

Sorry for using incorrect email address. I will use correct information to commit later.

@Akarinnnnn commented on GitHub (Dec 11, 2023): Sorry for using incorrect email address. I will use correct information to commit later.

claunia referenced this issue

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

[PR #649] [MERGED] Align Whitespace and Punctuation definitions with CommonMark #1164

Sign in to join this conversation.

Branches Tags

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: starred/markdig#645

Apply foreach on class StringLineGroup got phantom empty line, is it expected? #645

Apply `foreach` on `class StringLineGroup` got phantom empty line, is it expected? #645