Re: [Qemu-devel] [PATCH 11/13] tests: Don't call visit_end_struct() after visit_start_struct() fails

[Eric Blake]

On 05/02/2014 06:44 AM, Markus Armbruster wrote:
> When visit_start_struct() succeeds, visit_end_struct() must not be

As in 10/13, s/succeeds/fails/

> called.  Three out of four visit_type_TestStruct() call it anyway.  As
> far as I can tell, visit_start_struct() doesn't actually fail there.
> Fix them anyway.
> 
> Signed-off-by: Markus Armbruster <address@hidden>
> ---
>  tests/test-qmp-input-strict.c      | 18 +++++++++++++-----
>  tests/test-qmp-output-visitor.c    | 18 +++++++++++++-----
>  tests/test-visitor-serialization.c | 18 +++++++++++++-----
>  3 files changed, 39 insertions(+), 15 deletions(-)
> 

With commit message fixed,

Reviewed-by: Eric Blake <address@hidden>

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

signature.asc
Description: OpenPGP digital signature