[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V7 07/10] add def_print_str and use it in qemu_o
From: |
Kevin Wolf |
Subject: |
Re: [Qemu-devel] [PATCH V7 07/10] add def_print_str and use it in qemu_opts_print. |
Date: |
Tue, 11 Dec 2012 16:51:48 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:13.0) Gecko/20120605 Thunderbird/13.0 |
Am 06.12.2012 07:47, schrieb Dong Xu Wang:
> qemu_opts_print has no user now, so can re-write the function safely.
>
> qemu_opts_print will be used while using "qemu-img create", it will
> produce the same output as previous code.
>
> The behavior of this function has changed:
>
> 1. Print every possible option, whether a value has been set or not.
> 2. Option descriptors may provide a default value.
> 3. Print to stdout instead of stderr.
>
> Previously the behavior was to print every option that has been set.
> Options that have not been set would be skipped.
>
> Signed-off-by: Dong Xu Wang <address@hidden>
> ---
> qemu-option.c | 27 ++++++++++++++++++++-------
> qemu-option.h | 1 +
> 2 files changed, 21 insertions(+), 7 deletions(-)
>
> diff --git a/qemu-option.c b/qemu-option.c
> index 94557cf..e0628da 100644
> --- a/qemu-option.c
> +++ b/qemu-option.c
> @@ -862,15 +862,28 @@ void qemu_opts_del(QemuOpts *opts)
>
> int qemu_opts_print(QemuOpts *opts, void *dummy)
> {
> - QemuOpt *opt;
> + QemuOptDesc *desc = opts->list->desc;
>
> - fprintf(stderr, "%s: %s:", opts->list->name,
> - opts->id ? opts->id : "<noid>");
> - QTAILQ_FOREACH(opt, &opts->head, next) {
> - fprintf(stderr, " %s=\"%s\"", opt->name, opt->str);
I think we still need this old code path for the opts_accepts_any() ==
true case.
> diff --git a/qemu-option.h b/qemu-option.h
> index 002dd07..ab02023 100644
> --- a/qemu-option.h
> +++ b/qemu-option.h
> @@ -96,6 +96,7 @@ typedef struct QemuOptDesc {
> const char *name;
> enum QemuOptType type;
> const char *help;
> + const char *def_print_str;
> } QemuOptDesc;
Shouldn't we try to make this the real default value instead of just
what is printed if an option isn't set?
Kevin
- [Qemu-devel] [PATCH V7 00/10] replace QEMUOptionParameter with QemuOpts parser, Dong Xu Wang, 2012/12/06
- [Qemu-devel] [PATCH V7 01/10] qemu-option: opt_set(): split it up into more functions, Dong Xu Wang, 2012/12/06
- [Qemu-devel] [PATCH V7 03/10] qemu-option: qemu_opt_set_bool(): fix code duplication, Dong Xu Wang, 2012/12/06
- [Qemu-devel] [PATCH V7 02/10] qemu-option: qemu_opts_validate(): fix duplicated code, Dong Xu Wang, 2012/12/06
- [Qemu-devel] [PATCH V7 04/10] introduce qemu_opts_create_nofail function, Dong Xu Wang, 2012/12/06
- [Qemu-devel] [PATCH V7 06/10] create new function: qemu_opt_set_number, Dong Xu Wang, 2012/12/06
- [Qemu-devel] [PATCH V7 05/10] use qemu_opts_create_nofail, Dong Xu Wang, 2012/12/06
- [Qemu-devel] [PATCH V7 07/10] add def_print_str and use it in qemu_opts_print., Dong Xu Wang, 2012/12/06
- Re: [Qemu-devel] [PATCH V7 07/10] add def_print_str and use it in qemu_opts_print.,
Kevin Wolf <=
- [Qemu-devel] [PATCH V7 10/10] remove QEMUOptionParameter related functions and struct, Dong Xu Wang, 2012/12/06
- [Qemu-devel] [PATCH V7 08/10] Create four opts list related functions, Dong Xu Wang, 2012/12/06
- [Qemu-devel] [PATCH V7 09/10] Use QemuOpts support in block layer, Dong Xu Wang, 2012/12/06
- Re: [Qemu-devel] [PATCH V7 00/10] replace QEMUOptionParameter with QemuOpts parser, Stefan Hajnoczi, 2012/12/11
- Re: [Qemu-devel] [PATCH V7 00/10] replace QEMUOptionParameter with QemuOpts parser, Kevin Wolf, 2012/12/11