qemu-block
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-block] [Qemu-devel] [PATCH 01/34] qdict: Add qdict_array_entri


From: Eric Blake
Subject: Re: [Qemu-block] [Qemu-devel] [PATCH 01/34] qdict: Add qdict_array_entries()
Date: Fri, 08 May 2015 15:22:56 -0600
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.6.0

On 05/08/2015 02:06 PM, Eric Blake wrote:
> On 05/08/2015 11:21 AM, Kevin Wolf wrote:
>> Signed-off-by: Kevin Wolf <address@hidden>
>> ---
> 
> Might want to include mention of what it will be used for in the commit
> body.
> 
>>  include/qapi/qmp/qdict.h |  1 +
>>  qobject/qdict.c          | 68 
>> +++++++++++++++++++++++++++++++++++++++++++++---
>>  2 files changed, 65 insertions(+), 4 deletions(-)
>>

Oh, and I meant to mention: shouldn't the testsuite also be enhanced to
cover the new functionality?

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

Attachment: signature.asc
Description: OpenPGP digital signature


reply via email to

[Prev in Thread] Current Thread [Next in Thread]