[libvirt] [PATCH 02/19] test: Use consistent variable names for storage test driver APIs
Pavel Hrdina
phrdina at redhat.com
Tue Jul 11 08:27:46 UTC 2017
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 <jferlan at redhat.com>
> ---
> 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
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20170711/a51b3c9b/attachment-0001.sig>
More information about the libvir-list
mailing list