[libvirt] [PATCH 07/19] storage: Use consistent variable names in virstorageobj
Pavel Hrdina
phrdina at redhat.com
Wed Jul 12 08:17:33 UTC 2017
On Tue, May 09, 2017 at 11:30:14AM -0400, John Ferlan wrote:
> A virStoragePoolObjPtr will be an 'obj'.
>
> A virStoragePoolPtr will be a 'pool'.
There is no such change in this commit.
>
> NB: Also modify the @matchpool to @matchobj.
> Signed-off-by: John Ferlan <jferlan at redhat.com>
> ---
> src/conf/virstorageobj.c | 342 +++++++++++++++++++++++------------------------
> src/conf/virstorageobj.h | 20 +--
> 2 files changed, 180 insertions(+), 182 deletions(-)
>
> diff --git a/src/conf/virstorageobj.c b/src/conf/virstorageobj.c
> index dd41701..74a9c67 100644
> --- a/src/conf/virstorageobj.c
> +++ b/src/conf/virstorageobj.c
> @@ -70,15 +70,15 @@ virStoragePoolObjListFree(virStoragePoolObjListPtr pools)
>
> void
> virStoragePoolObjRemove(virStoragePoolObjListPtr pools,
> - virStoragePoolObjPtr pool)
> + virStoragePoolObjPtr obj)
This is why I'm not a fan of this change, to make the code consistent
we should also rename "pools" to "objs" which I don't like so probably
"objList" or something else.
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/20170712/c556076c/attachment-0001.sig>
More information about the libvir-list
mailing list