[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] qapi2texi: replace quotation by bold section na
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH] qapi2texi: replace quotation by bold section name |
Date: |
Mon, 20 Feb 2017 14:08:15 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/25.1 (gnu/linux) |
Marc-André Lureau <address@hidden> writes:
> When we build qemu-qmp-ref.txt this causes texinfo to complain several
> times:
> "Negative repeat count does nothing at
> /usr/share/texinfo/Texinfo/Convert/Line.pm line 124."
>
> It also doesn't display correctly, because the "Notes" text disappears
> entirely in the HTML version because it thinks there's no actual
> quotation text.
>
> The text file output formatting is also not good.
>
> To solve those problems, remove usage of @quotation, and simply use bold
> face for the section name.
>
> Reported-by: Peter Maydell <address@hidden>
> Signed-off-by: Marc-André Lureau <address@hidden>
> ---
> scripts/qapi2texi.py | 10 ++++------
> 1 file changed, 4 insertions(+), 6 deletions(-)
>
> diff --git a/scripts/qapi2texi.py b/scripts/qapi2texi.py
> index 83ded95c2d..c1071c62c6 100755
> --- a/scripts/qapi2texi.py
> +++ b/scripts/qapi2texi.py
> @@ -159,12 +159,10 @@ def texi_body(doc):
> func = texi_example
>
> if name:
> - # FIXME the indentation produced by @quotation in .txt and
> - # .html output is confusing
> - body += "address@hidden address@hidden quotation" % \
> - (name, func(doc))
> - else:
> - body += func(doc)
> + # prefer @b over @strong, so txt doesn't translate it to *Foo:*
> + body += "address@hidden:}\n" % name
The second newline is sometimes needed to make a paragraph, and at other
times it's redundant and somewhat ugly. Tolerable, as this file isn't
for humans.
> +
> + body += func(doc)
>
> return body
Reviewed-by: Markus Armbruster <address@hidden>