On Tue, May 09, 2017 at 11:30:09AM -0400, John Ferlan wrote: > A virStoragePoolObjPtr will be an 'obj'. > > A virStoragePoolPtr will be a 'pool'. > > Signed-off-by: John Ferlan <[email protected]> > --- > src/test/test_driver.c | 443 > ++++++++++++++++++++++++------------------------- > 1 file changed, 219 insertions(+), 224 deletions(-) > > diff --git a/src/test/test_driver.c b/src/test/test_driver.c > index 548f318..c0e46af 100644 > --- a/src/test/test_driver.c > +++ b/src/test/test_driver.c
[...]
> @@ -4057,18 +4056,18 @@ testStoragePoolLookupByUUID(virConnectPtr conn,
> const unsigned char *uuid)
> {
> testDriverPtr privconn = conn->privateData;
> - virStoragePoolObjPtr pool;
> + virStoragePoolObjPtr obj;
> virStoragePoolPtr ret = NULL;
There you didn't changed the "ret" to "pool".
> - if (!(pool = testStoragePoolObjFindByUUID(privconn, uuid)))
> + if (!(obj = testStoragePoolObjFindByUUID(privconn, uuid)))
> goto cleanup;
>
> - ret = virGetStoragePool(conn, pool->def->name, pool->def->uuid,
> + ret = virGetStoragePool(conn, obj->def->name, obj->def->uuid,
> NULL, NULL);
>
> cleanup:
> - if (pool)
> - virStoragePoolObjUnlock(pool);
> + if (obj)
> + virStoragePoolObjUnlock(obj);
> return ret;
> }
>
> @@ -4078,18 +4077,18 @@ testStoragePoolLookupByName(virConnectPtr conn,
> const char *name)
> {
> testDriverPtr privconn = conn->privateData;
> - virStoragePoolObjPtr pool;
> + virStoragePoolObjPtr obj;
> virStoragePoolPtr ret = NULL;
Same here.
> - if (!(pool = testStoragePoolObjFindByName(privconn, name)))
> + if (!(obj = testStoragePoolObjFindByName(privconn, name)))
> goto cleanup;
>
> - ret = virGetStoragePool(conn, pool->def->name, pool->def->uuid,
> + ret = virGetStoragePool(conn, obj->def->name, obj->def->uuid,
> NULL, NULL);
>
> cleanup:
> - if (pool)
> - virStoragePoolObjUnlock(pool);
> + if (obj)
> + virStoragePoolObjUnlock(obj);
> return ret;
> }
>
[...]
> @@ -4345,7 +4344,7 @@ testStoragePoolCreateXML(virConnectPtr conn,
> {
> testDriverPtr privconn = conn->privateData;
> virStoragePoolDefPtr def;
> - virStoragePoolObjPtr pool = NULL;
> + virStoragePoolObjPtr obj = NULL;
> virStoragePoolPtr ret = NULL;
And here.
> virObjectEventPtr event = NULL;
>
[...]
> @@ -4419,7 +4418,7 @@ testStoragePoolDefineXML(virConnectPtr conn,
> {
> testDriverPtr privconn = conn->privateData;
> virStoragePoolDefPtr def;
> - virStoragePoolObjPtr pool = NULL;
> + virStoragePoolObjPtr obj = NULL;
> virStoragePoolPtr ret = NULL;
And here
> virObjectEventPtr event = NULL;
>
[...]
I don't like these type of patches. The value to noise ration is poor.
I'm hesitating to give this patch an ACK even though I probably done
that in the past.
Pavel
signature.asc
Description: Digital signature
-- libvir-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/libvir-list
