Marc-André Lureau <marcandre.lur...@redhat.com> 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 <peter.mayd...@linaro.org>
> Signed-off-by: Marc-André Lureau <marcandre.lur...@redhat.com>
> ---
> 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 += "\n@quotation %s\n%s\n@end quotation" % \
> - (name, func(doc))
> - else:
> - body += func(doc)
> + # prefer @b over @strong, so txt doesn't translate it to *Foo:*
> + body += "\n\n@b{%s:}\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 <arm...@redhat.com>
