[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH v8 18/22] qapi: add 'If:' condition to enum valu

From: Markus Armbruster
Subject: Re: [Qemu-devel] [PATCH v8 18/22] qapi: add 'If:' condition to enum values documentation
Date: Thu, 13 Dec 2018 15:03:53 +0100
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/26.1 (gnu/linux)

Marc-André Lureau <address@hidden> writes:

> Use a common function to generate the "If:..." line.
> While at it, get rid of the existing \n\n (no idea why it was
> there). Use a line-break in member description, this seems to look
> slightly better in the plaintext version.
> Signed-off-by: Marc-André Lureau <address@hidden>

I'm not entirely happy with texi_member() (see my review of PATCH v7
22/27) and the way we present conditionals in generated documentation
(see my review of PATCH v7 26/27).  But this will do for now.

Reviewed-by: Markus Armbruster <address@hidden>

reply via email to

[Prev in Thread] Current Thread [Next in Thread]