[PATCH 0/7] qapi: Primarily whitespace tweaks

Andrea Bolognani posted 7 patches 1 year, 11 months ago
There is a newer version of this series
qapi/audio.json          |  1 -
qapi/block-core.json     | 97 ++++++++++++++++++----------------------
qapi/block-export.json   |  2 +-
qapi/block.json          | 13 +++---
qapi/char.json           | 10 ++---
qapi/common.json         |  2 -
qapi/control.json        | 13 +++---
qapi/crypto.json         | 62 ++++++++++++-------------
qapi/dump.json           |  4 +-
qapi/job.json            |  1 -
qapi/machine-target.json |  1 -
qapi/machine.json        | 12 +++--
qapi/migration.json      | 19 ++++----
qapi/misc-target.json    | 13 ++----
qapi/misc.json           |  6 +--
qapi/replay.json         |  1 -
qapi/run-state.json      | 10 ++---
qapi/sockets.json        |  6 +--
qapi/ui.json             | 70 ++++++++++-------------------
19 files changed, 136 insertions(+), 207 deletions(-)
[PATCH 0/7] qapi: Primarily whitespace tweaks
Posted by Andrea Bolognani 1 year, 11 months ago
The last patch could very reasonably be squashed into the previous
one, but since the changes could be considered more controversial I
thought it would be better if the two could be reviewed and judged
separately.

Andrea Bolognani (7):
  qapi: Drop stray trailing symbol
  qapi: Fix comment indentation
  qapi: Add missing separators between sections
  qapi: Drop unnecessary empty lines in comments
  qapi: Drop unnecessary empty lines outside of comments
  qapi: Drop unnecessary horizontal spacing in comments
  qapi: Drop more unnecessary horizontal spacing in comments

 qapi/audio.json          |  1 -
 qapi/block-core.json     | 97 ++++++++++++++++++----------------------
 qapi/block-export.json   |  2 +-
 qapi/block.json          | 13 +++---
 qapi/char.json           | 10 ++---
 qapi/common.json         |  2 -
 qapi/control.json        | 13 +++---
 qapi/crypto.json         | 62 ++++++++++++-------------
 qapi/dump.json           |  4 +-
 qapi/job.json            |  1 -
 qapi/machine-target.json |  1 -
 qapi/machine.json        | 12 +++--
 qapi/migration.json      | 19 ++++----
 qapi/misc-target.json    | 13 ++----
 qapi/misc.json           |  6 +--
 qapi/replay.json         |  1 -
 qapi/run-state.json      | 10 ++---
 qapi/sockets.json        |  6 +--
 qapi/ui.json             | 70 ++++++++++-------------------
 19 files changed, 136 insertions(+), 207 deletions(-)

-- 
2.35.1

Re: [PATCH 0/7] qapi: Primarily whitespace tweaks
Posted by Markus Armbruster 1 year, 11 months ago
PATCH 1-5 queued, because no-brainers :)
Re: [PATCH 0/7] qapi: Primarily whitespace tweaks
Posted by Andrea Bolognani 1 year, 11 months ago
On Mon, May 02, 2022 at 02:43:52PM +0200, Markus Armbruster wrote:
> PATCH 1-5 queued, because no-brainers :)

Thanks!

How do you want me to handle respinning 6/7 and 7/7? Send out the
entire series again with those two patches tweaked, wait for your
pull request to make it into the tree, something else... ?

What about the changes you suggested to the commit message of 3/7?

-- 
Andrea Bolognani / Red Hat / Virtualization
Re: [PATCH 0/7] qapi: Primarily whitespace tweaks
Posted by Markus Armbruster 1 year, 11 months ago
Andrea Bolognani <abologna@redhat.com> writes:

> On Mon, May 02, 2022 at 02:43:52PM +0200, Markus Armbruster wrote:
>> PATCH 1-5 queued, because no-brainers :)
>
> Thanks!
>
> How do you want me to handle respinning 6/7 and 7/7? Send out the
> entire series again with those two patches tweaked, wait for your
> pull request to make it into the tree, something else... ?

If you like, I can post a pull request tomorrow morning.

Respinning the complete series is fine with me.

> What about the changes you suggested to the commit message of 3/7?

If you respin the complete series, you get to can amend it.  Else I do.