On 01/08/2016 06:39 AM, Peter Krempa wrote:
On Thu, Jan 07, 2016 at 22:50:05 -0500, Cole Robinson wrote:
> Future changes will make some of these tests dependent on specific
> QEMUCaps flags, so wire up the basic handling.
> ---
> tests/qemuxml2xmltest.c | 11 +++++++++++
> 1 file changed, 11 insertions(+)
>
> diff --git a/tests/qemuxml2xmltest.c b/tests/qemuxml2xmltest.c
> index c0270d4..32c9fed 100644
> --- a/tests/qemuxml2xmltest.c
> +++ b/tests/qemuxml2xmltest.c
[...]
> @@ -225,6 +229,13 @@ testInfoSet(struct testInfo *info,
> bool different,
> int when)
> {
> + if (!(info->qemuCaps = virQEMUCapsNew()))
> + goto error;
This is not necessary, since ...
> +
> + if (qemuTestCapsCacheInsert(driver.qemuCapsCache, name,
> + info->qemuCaps) < 0)
> + goto error;
... this function allocates the caps if it's called with the third
argument set to NULL.
I didn't see that. However then we don't have a handle to actually set CAPS
flags on, we would have to look it up, which is roughly the same amount of code.
I reposted this patch unchanged in my latest series
Thanks,
Cole