[libvirt] [PATCH 15/16] network: Modify naming for virNetworkObjList* fetching APIs

Pavel Hrdina phrdina at redhat.com
Mon Jul 24 11:49:13 UTC 2017


On Fri, May 19, 2017 at 09:03:23AM -0400, John Ferlan wrote:
> Use the structure names in the @data setup - makes it easier that going
> back to find the struct.
>
> Use the @maxnames instead of @nnames since that's what it is.

Please use camelCase -> @maxNames.

> 
> Modify the @filter to be @aclfilter and change the typedef from
> virNetworkObjListFilter to virNetworkObjListACLFilter.

NACK to this change, even though it's used only to filter by ACLs, it
can be used to filter by anything.

This patch does three things in one, so it should be three separate
patches.  Since the last change is not correct split the remaining
changes into two patches.

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/20170724/3f1cb90e/attachment-0001.sig>


More information about the libvir-list mailing list