[libvirt] [PATCH 2/2]: Remove getURI internal driver call

Daniel P. Berrange berrange at redhat.com
Tue Mar 31 09:54:31 UTC 2009


On Tue, Mar 31, 2009 at 11:41:58AM +0200, Daniel Veillard wrote:
> On Mon, Mar 30, 2009 at 02:10:13PM +0200, Chris Lalancette wrote:
> > This patch removes the internal getURI() driver callback.  It was *never*
> > overridden by any of the drivers, so it seems kind of pointless (I even checked
> > the "in-progress" VirtualBox, OpenNebula, and Power Hypervisor drivers).  In
> > addition, the code to actually use it in src/libvirt.c:virConnectGetURI() was
> > buggy; if a driver did decide to have their own callback, then their callback
> > would be called, and then immediately overwritten by the generic stuff.  This
> > wouldn't do what the driver expected, and would also probably leak memory.  Just
> > rip out this internal callback; if a future driver ever needs it, we can add it
> > back in.
> 
>   Agreed if the entry point is not used let's remove it, ACK !

ACK. It was used in the past, but I guess I made it unneccessary when I
patched virConnectPtr to include a full xmlURIPtr object.

Daniel
-- 
|: Red Hat, Engineering, London   -o-   http://people.redhat.com/berrange/ :|
|: http://libvirt.org  -o-  http://virt-manager.org  -o-  http://ovirt.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: GnuPG: 7D3B9505  -o-  F3C9 553F A1DA 4AC2 5648 23C1 B3DF F742 7D3B 9505 :|




More information about the libvir-list mailing list