Thank you very much! This looks good.

Interestingly/weirdly, ImDrawListSplitter::Merge() has similar code with an explicit comment:

// Equivalent of PopUnusedDrawCmd() for this channel's cmdbuffer and except we don't need to test for UserCallback.
if (ch._CmdBuffer.Size > 0 && ch._CmdBuffer.back().ElemCount == 0)
    ch._CmdBuffer.pop_back();

I am confused as to what why we did that.... git blaming...

Added the comment in 117d57d while refactoring but after the fact of writing the code like that... Actual logic comes from the very old 87ebe95. My opinion is that logic was wrong in the first place.

As a result I have amended your commit with the same change in ImDrawListSplitter::Merge() as well.

Thank you.

Appreciate the coauthor! I did see it in ImDrawListSplitter::Merge() but assumed there must have been a reason for it since it was mentioned that ".. we don't need to test for UserCallback..".

Either way, glad to have my first small commit!

The PR was wrong, we didn't noticed as other flags which led to additional drawing (BordersH, RowBG) would strip that empty draw command in other places. Discovered by @rokups.
Pushed fix f58bd81 + amended our tests.