[libvirt] [PATCH v1 24/31] bridge_driver: Use virNetworkObjEndAPI

Peter Krempa pkrempa at redhat.com
Tue Mar 3 13:40:21 UTC 2015


On Thu, Feb 26, 2015 at 15:17:33 +0100, Michal Privoznik wrote:
> So far, this is pure code replacement. But once we introduce
> reference counting to virNetworkObj this will be more handy as
> there'll be only one function to change: virNetworkObjEndAPI().
> 
> Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
> ---
>  src/network/bridge_driver.c | 57 +++++++++++++++------------------------------
>  1 file changed, 19 insertions(+), 38 deletions(-)

In the qemu driver that is the inspiration for this change we always
pair the *EndAPI call with a  *ObjFrom* ... in this case
networkObjFromNetwork.

I'd really appreciate if you could first scrub the code base of direct
lookup and use the networkObjFromNetwork helper instead so that we can
pair every EndAPI call with that.

ACK to this patch though. If you don't fancy cleaning the existing
places that manually lookup the object I can do it. Please let me know.

Peter
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 819 bytes
Desc: Digital signature
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20150303/cdce8369/attachment-0001.sig>


More information about the libvir-list mailing list