================ @@ -352,40 +606,59 @@ TEST(BulletList, Render) { DeepDoc.addParagraph().appendText("baz"); StringRef ExpectedMarkdown = R"md(- foo - bar -- foo - baz - - foo - - baz +- foo + ---------------- tcottin wrote:
Yes. We have to add them because we now end a bullet list with 2 new lines to have proper separation of bullet lists and normal paragraphs. And I could not find a solution to do it differently. We would need to know before we finish a bullet list if another bullet list follows. And if another follows only add a single new line. At least thats what I think we would need to solve. Maybe you have another suggestion/idea? Besides that, I dont think it is a big issue: 1. It is only an issue if nested bullet lists are defined by a user outside of the documentation parsing. Currently, I did not see nested bullet lists used outside of this test. 2. For the documentation markup, nested bullet lists are also not used. And for the future doxygen parsing, [nested bullet lists are also not supported](https://www.doxygen.nl/manual/commands.html#cmdli). 3. This does not affect lists which are written as markdown in the documentation. https://github.com/llvm/llvm-project/pull/140498 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits