[libvirt] [PATCH 09/20] qemumonitortestutils: Split up creation of the test to allow reuse

Eric Blake eblake at redhat.com
Tue Jul 30 14:00:31 UTC 2013


On 07/30/2013 07:05 AM, Peter Krempa wrote:
> The instrumentation for the monitor test can be hacked for qemu agent
> testing. Split out the monitor specific stuff to allow using the code in
> guest agent tests in the future.
> ---
>  tests/qemumonitortestutils.c | 96 ++++++++++++++++++++++++++++++--------------
>  1 file changed, 66 insertions(+), 30 deletions(-)
> 
> diff --git a/tests/qemumonitortestutils.c b/tests/qemumonitortestutils.c
> index 5ca569f..1785293 100644
> --- a/tests/qemumonitortestutils.c
> +++ b/tests/qemumonitortestutils.c
> @@ -440,16 +440,11 @@ static qemuMonitorCallbacks qemuCallbacks = {
>  };
> 
> 
> -#define QEMU_JSON_GREETING "{\"QMP\": {\"version\": {\"qemu\": {\"micro\": 1, \"minor\": 0, \"major\": 1}, \"package\": \" (qemu-kvm-1.0.1)\"}, \"capabilities\": []}}"

Ah, as long as you are refactoring here, you don't have to tweak the
earlier patch in the series where I suggested breaking the long line.

> 
> +#define QEMU_JSON_GREETING "{\"QMP\": {\"version\": {\"qemu\": {\"micro\": 1, \"minor\": 0, \"major\": 1}, \"package\": \" (qemu-kvm-1.0.1)\"}, \"capabilities\": []}}"

But here, you SHOULD break up the long line :)

ACK with that fixed.

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

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 621 bytes
Desc: OpenPGP digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20130730/f58731a5/attachment-0001.sig>


More information about the libvir-list mailing list