[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v3 05/12] scripts/qapi/parser.py: improve doc comment indent
From: |
Richard Henderson |
Subject: |
Re: [PATCH v3 05/12] scripts/qapi/parser.py: improve doc comment indent handling |
Date: |
Thu, 27 Feb 2020 06:13:27 -0800 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.4.1 |
On 2/25/20 6:04 AM, Peter Maydell wrote:
> Make the handling of indentation in doc comments more sophisticated,
> so that when we see a section like:
>
> Notes: some text
> some more text
> indented line 3
>
> we save it for the doc-comment processing code as:
>
> some text
> some more text
> indented line 3
>
> and when we see a section with the heading on its own line:
>
> Notes:
>
> some text
> some more text
> indented text
>
> we also accept that and save it in the same form.
>
> The exception is that we always retain indentation as-is for Examples
> sections, because these are literal text.
>
> If we detect that the comment document text is not indented as much
> as we expect it to be, we throw a parse error. (We don't complain
> about over-indented sections, because for rST this can be legitimate
> markup.)
>
> The golden reference for the doc comment text is updated to remove
> the two 'wrong' indents; these now form a test case that we correctly
> stripped leading whitespace from an indented multi-line argument
> definition.
>
> Signed-off-by: Peter Maydell <address@hidden>
> ---
> v1->v2: Update doc-good.out as per final para.
> ---
> scripts/qapi/parser.py | 82 +++++++++++++++++++++++++++-------
> tests/qapi-schema/doc-good.out | 4 +-
> 2 files changed, 67 insertions(+), 19 deletions(-)
Reviewed-by: Richard Henderson <address@hidden>
r~
- [PATCH v3 00/12] Convert QAPI doc comments to generate rST instead of texinfo, Peter Maydell, 2020/02/25
- [PATCH v3 01/12] qapi/qapi-schema.json: Put headers in their own doc-comment blocks, Peter Maydell, 2020/02/25
- [PATCH v3 02/12] qapi/machine.json: Escape a literal '*' in doc comment, Peter Maydell, 2020/02/25
- [PATCH v3 03/12] tests/qapi/doc-good.json: Clean up markup, Peter Maydell, 2020/02/25
- [PATCH v3 04/12] scripts/qapi: Move doc-comment whitespace stripping to doc.py, Peter Maydell, 2020/02/25
- [PATCH v3 05/12] scripts/qapi/parser.py: improve doc comment indent handling, Peter Maydell, 2020/02/25
- Re: [PATCH v3 05/12] scripts/qapi/parser.py: improve doc comment indent handling,
Richard Henderson <=
- [PATCH v3 09/12] qapi: Use rST markup for literal blocks, Peter Maydell, 2020/02/25
- [PATCH v3 06/12] docs/sphinx: Add new qapi-doc Sphinx extension, Peter Maydell, 2020/02/25
- [PATCH v3 07/12] docs/interop: Convert qemu-ga-ref to rST, Peter Maydell, 2020/02/25
- [PATCH v3 10/12] qga/qapi-schema.json: Add some headings, Peter Maydell, 2020/02/25
- [PATCH v3 08/12] docs/interop: Convert qemu-qmp-ref to rST, Peter Maydell, 2020/02/25
- [PATCH v3 12/12] docs/devel/qapi-code-gen.txt: Update to new rST backend conventions, Peter Maydell, 2020/02/25
- [PATCH v3 11/12] scripts/qapi: Remove texinfo generation support, Peter Maydell, 2020/02/25
- Re: [PATCH v3 00/12] Convert QAPI doc comments to generate rST instead of texinfo, Markus Armbruster, 2020/02/26