qemu-devel
[Top][All Lists]
Advanced

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

Re: [Qemu-devel] [PATCH 3/3] check-qdict: Adjust test for qdict_array_sp


From: Max Reitz
Subject: Re: [Qemu-devel] [PATCH 3/3] check-qdict: Adjust test for qdict_array_split()
Date: Fri, 21 Feb 2014 20:41:49 +0100
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:24.0) Gecko/20100101 Thunderbird/24.3.0

On 21.02.2014 19:39, Eric Blake wrote:
On 02/21/2014 11:11 AM, Max Reitz wrote:
Test the new functionality of qdict_array_split(), that is, splitting
off single objects.

Signed-off-by: Max Reitz <address@hidden>
---
  tests/check-qdict.c | 22 ++++++++++++++++------
  1 file changed, 16 insertions(+), 6 deletions(-)
Reviewed-by: Eric Blake <address@hidden>

However, I think it is incomplete - you didn't test the behavior when
both %u and %u.sub appear in the same dict, to make sure the splitting
stops at that point.

Yes, I'll add a test for that case, too.


Thanks for you review,

Max



reply via email to

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