All of lore.kernel.org
 help / color / mirror / Atom feed
From: Eric Blake <eblake@redhat.com>
To: Chunyan Liu <cyliu@suse.com>, qemu-devel@nongnu.org
Cc: kwolf@redhat.com, Dong Xu Wang <wdongxu@linux.vnet.ibm.com>,
	stefanha@redhat.com
Subject: Re: [Qemu-devel] [PATCH v21 02/25] qapi: output def_value_str when query command line options
Date: Mon, 03 Mar 2014 17:02:27 -0700	[thread overview]
Message-ID: <53151813.1070104@redhat.com> (raw)
In-Reply-To: <1392978948-27416-3-git-send-email-cyliu@suse.com>

[-- Attachment #1: Type: text/plain, Size: 1194 bytes --]

On 02/21/2014 03:35 AM, Chunyan Liu wrote:
> Change qapi interfaces to output the newly added def_value_str when querying
> command line options.
> 
> Signed-off-by: Dong Xu Wang <wdongxu@linux.vnet.ibm.com>
> Signed-off-by: Chunyan Liu <cyliu@suse.com>
> ---
>  qapi-schema.json   |    8 ++++++--
>  qmp-commands.hx    |    2 ++
>  util/qemu-config.c |    4 ++++
>  3 files changed, 12 insertions(+), 2 deletions(-)
> 
> diff --git a/qapi-schema.json b/qapi-schema.json
> index 473c096..83e5870 100644
> --- a/qapi-schema.json
> +++ b/qapi-schema.json
> @@ -4010,12 +4010,16 @@
>  #
>  # @help: #optional human readable text string, not suitable for parsing.
>  #
> -# Since 1.5
> +# @default: #optional string representation of the default used
> +#           if the option is omitted.
> +#
> +# Since 2.0

Not quite right.  This needs to be:

# @help: #optional human readable text string, not suitable for parsing.
#
# @default: #optional string representation of the default used
#           if the option is omitted (since 2.0).
#
# Since 1.5

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 604 bytes --]

  reply	other threads:[~2014-03-04  0:02 UTC|newest]

Thread overview: 28+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2014-02-21 10:35 [Qemu-devel] [PATCH v21 00/25] replace QEMUOptionParameter with QemuOpts Chunyan Liu
2014-02-21 10:35 ` [Qemu-devel] [PATCH v21 01/25] add def_value_str to QemuOptDesc Chunyan Liu
2014-02-21 10:35 ` [Qemu-devel] [PATCH v21 02/25] qapi: output def_value_str when query command line options Chunyan Liu
2014-03-04  0:02   ` Eric Blake [this message]
2014-03-04  2:01     ` Chunyan Liu
2014-02-21 10:35 ` [Qemu-devel] [PATCH v21 03/25] improve some functions in qemu-option.c Chunyan Liu
2014-02-21 10:35 ` [Qemu-devel] [PATCH v21 04/25] improve assertion in qemu_opt_get functions Chunyan Liu
2014-02-21 10:35 ` [Qemu-devel] [PATCH v21 05/25] add some QemuOpts functions for replace work Chunyan Liu
2014-02-21 10:35 ` [Qemu-devel] [PATCH v21 06/25] add convert functions between QEMUOptionParameter to QemuOpts Chunyan Liu
2014-02-21 10:35 ` [Qemu-devel] [PATCH v21 07/25] change block layer to support both QemuOpts and QEMUOptionParamter Chunyan Liu
2014-02-21 10:35 ` [Qemu-devel] [PATCH v21 08/25] cow.c: replace QEMUOptionParameter with QemuOpts Chunyan Liu
2014-02-21 10:35 ` [Qemu-devel] [PATCH v21 09/25] gluster.c: " Chunyan Liu
2014-02-21 10:35 ` [Qemu-devel] [PATCH v21 10/25] iscsi.c: " Chunyan Liu
2014-02-21 10:35 ` [Qemu-devel] [PATCH v21 11/25] qcow.c: " Chunyan Liu
2014-02-21 10:35 ` [Qemu-devel] [PATCH v21 12/25] qcow2.c: " Chunyan Liu
2014-02-21 10:35 ` [Qemu-devel] [PATCH v21 13/25] qed.c: " Chunyan Liu
2014-02-21 10:35 ` [Qemu-devel] [PATCH v21 14/25] raw-posix.c: " Chunyan Liu
2014-02-21 10:35 ` [Qemu-devel] [PATCH v21 15/25] raw-win32.c: " Chunyan Liu
2014-02-21 10:35 ` [Qemu-devel] [PATCH v21 16/25] raw_bsd.c: " Chunyan Liu
2014-02-21 10:35 ` [Qemu-devel] [PATCH v21 17/25] rbd.c: " Chunyan Liu
2014-02-21 10:35 ` [Qemu-devel] [PATCH v21 18/25] sheepdog.c: " Chunyan Liu
2014-02-21 10:35 ` [Qemu-devel] [PATCH v21 19/25] ssh.c: " Chunyan Liu
2014-02-21 10:35 ` [Qemu-devel] [PATCH v21 20/25] vdi.c: " Chunyan Liu
2014-02-21 10:35 ` [Qemu-devel] [PATCH v21 21/25] vmdk.c: " Chunyan Liu
2014-02-21 10:35 ` [Qemu-devel] [PATCH v21 22/25] vpc.c: " Chunyan Liu
2014-02-21 10:35 ` [Qemu-devel] [PATCH v21 23/25] vhdx.c: " Chunyan Liu
2014-02-21 10:35 ` [Qemu-devel] [PATCH v21 24/25] vvfat.c: " Chunyan Liu
2014-02-21 10:35 ` [Qemu-devel] [PATCH v21 25/25] cleanup QEMUOptionParameter Chunyan Liu

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=53151813.1070104@redhat.com \
    --to=eblake@redhat.com \
    --cc=cyliu@suse.com \
    --cc=kwolf@redhat.com \
    --cc=qemu-devel@nongnu.org \
    --cc=stefanha@redhat.com \
    --cc=wdongxu@linux.vnet.ibm.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.