On Wed, Mar 27, 2013 at 01:07:56PM +0800, Li Zhang wrote:
> From: Li Zhang <zhlci...@linux.vnet.ibm.com>
> 
> This patch is to add NVRAM test cases.
> 
> Signed-off-by: Li Zhang <zhlci...@linux.vnet.ibm.com>
> ---
>  .../qemuxml2argv-pseries-nvram.args                |    1 +
>  .../qemuxml2argv-pseries-nvram.xml                 |   20 
> ++++++++++++++++++++
>  tests/qemuxml2argvtest.c                           |    2 ++

You should also add the same data files in tests/qemuargv2xmltest.c to validate
your code for parsing ARGV.


Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Reply via email to