[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v3 06/14] qapi: fix various symbols mismatch in
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v3 06/14] qapi: fix various symbols mismatch in documentation |
Date: |
Mon, 07 Nov 2016 16:40:20 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Marc-André Lureau <address@hidden> writes:
> There are various mismatch:
> - invalid symbols
> - section and member symbols mismatch
> - enum or union values vs 'type'
>
> The documentation parser catches all these cases.
>
> Signed-off-by: Marc-André Lureau <address@hidden>
> ---
> qapi-schema.json | 20 +++++++++-----------
> qapi/block-core.json | 4 ----
> qapi/common.json | 6 +++---
> qapi/rocker.json | 2 +-
> qga/qapi-schema.json | 6 +++---
> 5 files changed, 16 insertions(+), 22 deletions(-)
>
> diff --git a/qapi-schema.json b/qapi-schema.json
> index 5f377fa..0c9a293 100644
> --- a/qapi-schema.json
> +++ b/qapi-schema.json
> @@ -693,8 +693,6 @@
> #
> # Set various migration parameters. See MigrationParameters for details.
> #
> -# @x-checkpoint-delay: the delay time between two checkpoints. (Since 2.8)
> -#
> # Since: 2.4
> ##
> { 'command': 'migrate-set-parameters', 'boxed': true,
> @@ -1171,7 +1169,7 @@
> '*service': 'str', '*auth': 'str', '*clients': ['VncClientInfo']}
> }
>
> ##
> -# @VncPriAuth:
> +# @VncPrimaryAuth:
> #
> # vnc primary authentication method.
> #
> @@ -3174,7 +3172,7 @@
> #
> # @alias: #optional an alias for the machine name
> #
> -# @default: #optional whether the machine is default
> +# @is-default: #optional whether the machine is default
> #
> # @cpu-max: maximum number of CPUs supported by the machine type
> # (since 1.5.0)
> @@ -3727,7 +3725,6 @@
> #
> # @device: The name of the special file for the device,
> # i.e. /dev/ttyS0 on Unix or COM1: on Windows
> -# @type: What kind of device this is.
> #
> # Since: 1.4
> ##
> @@ -3992,7 +3989,7 @@
> #
> # A union referencing different TPM backend types' configuration options
> #
> -# @passthrough: The configuration options for the TPM passthrough type
> +# @type: 'passthrough' The configuration options for the TPM passthrough type
> #
> # Since: 1.5
> ##
> @@ -4000,7 +3997,7 @@
> 'data': { 'passthrough' : 'TPMPassthroughOptions' } }
>
> ##
> -# @TpmInfo:
> +# @TPMInfo:
> #
> # Information about the TPM
> #
> @@ -4344,10 +4341,11 @@
> #
> # Input event union.
> #
> -# @key: Input event of Keyboard
> -# @btn: Input event of pointer buttons
> -# @rel: Input event of relative pointer motion
> -# @abs: Input event of absolute pointer motion
> +# @type: the input type, one of:
> +# - 'key': Input event of Keyboard
> +# - 'btn': Input event of pointer buttons
> +# - 'rel': Input event of relative pointer motion
> +# - 'abs': Input event of absolute pointer motion
> #
> # Since: 2.0
> ##
This will do for now, but we need to figure out how to document unions
properly.
> diff --git a/qapi/block-core.json b/qapi/block-core.json
> index 372889c..c64a48c 100644
> --- a/qapi/block-core.json
> +++ b/qapi/block-core.json
> @@ -2160,10 +2160,6 @@
> #
> # @type: Transport type used for gluster connection
> #
> -# @unix: socket file
> -#
> -# @tcp: host address and port number
> -#
> # This is similar to SocketAddress, only distinction:
> #
> # 1. GlusterServer is a flat union, SocketAddress is a simple union.
Unions again. We lose a bit of information here, but it's almost
worthless information.
> diff --git a/qapi/common.json b/qapi/common.json
> index 9353a7b..6987100 100644
> --- a/qapi/common.json
> +++ b/qapi/common.json
> @@ -34,11 +34,11 @@
> #
> # A three-part version number.
> #
> -# @qemu.major: The major version number.
> +# @major: The major version number.
> #
> -# @qemu.minor: The minor version number.
> +# @minor: The minor version number.
> #
> -# @qemu.micro: The micro version number.
> +# @micro: The micro version number.
> #
> # Since: 2.4
> ##
> diff --git a/qapi/rocker.json b/qapi/rocker.json
> index 2fe7fdf..ace2776 100644
> --- a/qapi/rocker.json
> +++ b/qapi/rocker.json
> @@ -1,5 +1,5 @@
> ##
> -# @Rocker:
> +# @RockerSwitch:
> #
> # Rocker switch information.
> #
> diff --git a/qga/qapi-schema.json b/qga/qapi-schema.json
> index 5173c4a..2e6cc91 100644
> --- a/qga/qapi-schema.json
> +++ b/qga/qapi-schema.json
> @@ -203,7 +203,7 @@
> #
> # Open a file in the guest and retrieve a file handle for it
> #
> -# @filepath: Full path to the file in the guest to open.
> +# @path: Full path to the file in the guest to open.
> #
> # @mode: #optional open mode, as per fopen(), "r" is the default.
> #
> @@ -378,7 +378,7 @@
> 'data': { 'handle': 'int' } }
>
> ##
> -# @GuestFsFreezeStatus
> +# @GuestFsfreezeStatus
> #
> # An enumeration of filesystem freeze states
> #
> @@ -766,7 +766,7 @@
> # @GuestDiskAddress:
> #
> # @pci-controller: controller's PCI address
> -# @type: bus type
> +# @bus-type: bus type
> # @bus: bus id
> # @target: target id
> # @unit: unit id
Lovely doc fixes :)
- Re: [Qemu-devel] [PATCH v3 01/14] qapi: add missing 'bus' argument in device_add, (continued)
- [Qemu-devel] [PATCH v3 02/14] qga/schema: fix double-return in doc, Marc-André Lureau, 2016/11/07
- [Qemu-devel] [PATCH v3 03/14] qga/schema: improve guest-set-vcpus Returns: section, Marc-André Lureau, 2016/11/07
- [Qemu-devel] [PATCH v3 05/14] qapi: fix missing symbol @prefix, Marc-André Lureau, 2016/11/07
- [Qemu-devel] [PATCH v3 04/14] qapi: fix schema symbol sections, Marc-André Lureau, 2016/11/07
- [Qemu-devel] [PATCH v3 06/14] qapi: fix various symbols mismatch in documentation, Marc-André Lureau, 2016/11/07
- Re: [Qemu-devel] [PATCH v3 06/14] qapi: fix various symbols mismatch in documentation,
Markus Armbruster <=
- [Qemu-devel] [PATCH v3 07/14] qapi: use one symbol per line, Marc-André Lureau, 2016/11/07
- [Qemu-devel] [PATCH v3 09/14] qapi: add some sections in docs, Marc-André Lureau, 2016/11/07
- [Qemu-devel] [PATCH v3 10/14] docs: add master qapi texi files, Marc-André Lureau, 2016/11/07
- [Qemu-devel] [PATCH v3 08/14] qapi: add missing colon-ending for section name, Marc-André Lureau, 2016/11/07
- [Qemu-devel] [PATCH v3 12/14] texi2pod: learn quotation, deftp and deftypefn, Marc-André Lureau, 2016/11/07
- [Qemu-devel] [PATCH v3 11/14] qapi: add qapi2texi script, Marc-André Lureau, 2016/11/07