amoeba commented on PR #41411: URL: https://github.com/apache/arrow/pull/41411#issuecomment-2083864439
Good eye @kou. Re: > "i., ii., ..." style and "1., 2., ..." style are mixed (Can we control `<ol>` style?) It doesn't look like it. The Doxygen output is what I expect but it looks like Breathe is getting an incorrect nesting level. [Their code](https://github.com/breathe-doc/breathe/blob/542ae9b9ed6e5bd5e92ba2360b094df8931ba756/breathe/renderer/sphinxrenderer.py#L1811-L1812) is pretty easy to follow except for how the nesting level ends up higher than what we expect. Their docs give some examples and, if you look at the final one in https://breathe.readthedocs.io/en/latest/lists.html, it actually looks wrong to me. So maybe this is by design? I don't see any config options we can use to control this. I opted to refer to list items as (1), (2), etc instead of (i), (ii), etc so that the comment is correct on its own and accept that the rendered docs are quite right. > Can we remove needless \ from ...\<account>... in ii.? Done. I ended up having to escape the `@` which is unfortunate but Doxygen is handling it strangely, see my commit message. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
