[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 08/29] qapi: Use ':' after @argument in doc comments
From: |
Kevin Wolf |
Subject: |
Re: [PATCH 08/29] qapi: Use ':' after @argument in doc comments |
Date: |
Fri, 7 Feb 2020 11:24:33 +0100 |
User-agent: |
Mutt/1.12.1 (2019-06-15) |
Am 07.02.2020 um 10:28 hat Markus Armbruster geschrieben:
> Peter Maydell <address@hidden> writes:
> > @@ -4703,7 +4703,7 @@
> > #
> > # Options for creating an image format on a given node.
> > #
> > -# @driver block driver to create the image format
> > +# @driver: block driver to create the image format
> > #
> > # Since: 2.12
> > ##
>
> Loses the visual alignment. I'm okay with that, but the folks who took
> the trouble to align the text may have different ideas. Cc'ing Kevin
> and Max.
I think the documentation is much easier to parse visually with aligned
text as it makes both the option name and the whole part of the comment
that documents options stand out clearly.
Of course, "It is the QEMU coding style." would trump everything, but as
long as there isn't a style guide that requires a wall of text without
spaces and alignment, I'd prefer to leave at least those aligned texts
in place that we have.
Kevin
- [PATCH 06/29] qga/qapi-schema.json: minor format fixups for rST, (continued)
- [PATCH 07/29] qapi/block-core.json: Use literal block for ascii art, Peter Maydell, 2020/02/06
- [PATCH 08/29] qapi: Use ':' after @argument in doc comments, Peter Maydell, 2020/02/06
- Re: [PATCH 08/29] qapi: Use ':' after @argument in doc comments, Markus Armbruster, 2020/02/07
- Re: [PATCH 08/29] qapi: Use ':' after @argument in doc comments, Max Reitz, 2020/02/07
- Re: [PATCH 08/29] qapi: Use ':' after @argument in doc comments,
Kevin Wolf <=
- Re: [PATCH 08/29] qapi: Use ':' after @argument in doc comments, Peter Maydell, 2020/02/07
- Re: [PATCH 08/29] qapi: Use ':' after @argument in doc comments, Markus Armbruster, 2020/02/07
- Re: [PATCH 08/29] qapi: Use ':' after @argument in doc comments, Max Reitz, 2020/02/07
- Re: [PATCH 08/29] qapi: Use ':' after @argument in doc comments, Kevin Wolf, 2020/02/07
- Re: [PATCH 08/29] qapi: Use ':' after @argument in doc comments, Peter Maydell, 2020/02/07
- Re: [PATCH 08/29] qapi: Use ':' after @argument in doc comments, Markus Armbruster, 2020/02/08
- Re: [PATCH 08/29] qapi: Use ':' after @argument in doc comments, Peter Maydell, 2020/02/08
[PATCH 11/29] qapi/ui.json: Put input-send-event body text in the right place, Peter Maydell, 2020/02/06
[PATCH 13/29] qapi/ui.json: Avoid `...' texinfo style quoting, Peter Maydell, 2020/02/06