[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 01/14] qapi: BlockExportRemoveMode: move comments to TODO
From: |
John Snow |
Subject: |
Re: [PATCH 01/14] qapi: BlockExportRemoveMode: move comments to TODO |
Date: |
Thu, 24 Mar 2022 16:34:42 -0400 |
On Thu, Mar 24, 2022 at 1:50 PM Victor Toso <victortoso@redhat.com> wrote:
>
> @hide and @soft are potential additions which fits the TODO section
> perfectly.
>
> The main motivation is to avoid this whole block of comment entering
> the wrong section in the python parser.
>
> Signed-off-by: Victor Toso <victortoso@redhat.com>
> ---
> qapi/block-export.json | 10 +++++-----
> 1 file changed, 5 insertions(+), 5 deletions(-)
>
> diff --git a/qapi/block-export.json b/qapi/block-export.json
> index f183522d0d..1e34927f85 100644
> --- a/qapi/block-export.json
> +++ b/qapi/block-export.json
> @@ -219,13 +219,13 @@
> #
> # @hard: Drop all connections immediately and remove export.
> #
> -# Potential additional modes to be added in the future:
> +# TODO: Potential additional modes to be added in the future:
> #
> -# hide: Just hide export from new clients, leave existing connections as is.
> -# Remove export after all clients are disconnected.
> +# hide: Just hide export from new clients, leave existing connections
> as is.
> +# Remove export after all clients are disconnected.
> #
> -# soft: Hide export from new clients, answer with ESHUTDOWN for all further
> -# requests from existing clients.
> +# soft: Hide export from new clients, answer with ESHUTDOWN for all
> further
> +# requests from existing clients.
> #
> # Since: 2.12
> ##
> --
> 2.35.1
>
Does this help with something in particular? (Got an example for me?)
--js
[PATCH 04/14] qapi: fix example of BLOCK_JOB_PENDING event, Victor Toso, 2022/03/24
[PATCH 09/14] qapi: run-state examples: add missing member, Victor Toso, 2022/03/24