[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH V16 0/7] replace QEMUOptionParameter with QemuOp
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH V16 0/7] replace QEMUOptionParameter with QemuOpts parser |
Date: |
Tue, 09 Jul 2013 14:41:46 -0600 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130625 Thunderbird/17.0.7 |
On 07/04/2013 06:52 AM, Stefan Hajnoczi wrote:
> On Tue, Jun 18, 2013 at 05:31:52PM +0800, Dong Xu Wang wrote:
>> These patches will replace QEMUOptionParameter with QemuOpts. Change logs
>> please go to each patch's commit message.
>>
>> Dong Xu Wang (7):
>> add def_value_str in QemuOptDesc struct and rewrite qemu_opts_print
>> avoid duplication of default value in QemuOpts
>> Create four QemuOptsList related functions
>> Create some QemuOpts functons
>> Use QemuOpts support in block layer
>> query-command-line-options outputs def_value_str
>> remove QEMUOptionParameter related functions and struct
>>
>> V15->V16:
>> 1) discard double-initialization.
>> 2) use pointer directly, not g_strdup.
>> 3) modify query-command-line-options related code.
>> V14->V15:
>> 1) Only delete enum QEMUOptionParType.
>
> eblake: You commented on the last revision. Are you happy with v16?
Apologies for my delays (still digging myself out of 3 weeks
accumulation of mail after my vacation); I'll review within 24 hours.
--
Eric Blake eblake redhat com +1-919-301-3266
Libvirt virtualization library http://libvirt.org
signature.asc
Description: OpenPGP digital signature