[libvirt] [libvirt-glib] gobject: Remove redundant virtual functions

Daniel P. Berrange berrange at redhat.com
Fri Jul 3 14:35:41 UTC 2015


On Fri, Jul 03, 2015 at 03:17:42PM +0100, Zeeshan Ali (Khattak) wrote:
> These virtual functions were most likely a result of copy&paste error.
> ---
>  libvirt-gobject/libvirt-gobject-network.h | 5 +----
>  1 file changed, 1 insertion(+), 4 deletions(-)
> 
> diff --git a/libvirt-gobject/libvirt-gobject-network.h b/libvirt-gobject/libvirt-gobject-network.h
> index 5617ed6..8c31af4 100644
> --- a/libvirt-gobject/libvirt-gobject-network.h
> +++ b/libvirt-gobject/libvirt-gobject-network.h
> @@ -55,10 +55,7 @@ struct _GVirNetworkClass
>  {
>      GObjectClass parent_class;
>  
> -    void (*started)(GVirNetwork *net);
> -    void (*stopped)(GVirNetwork *net);
> -
> -    gpointer padding[20];
> +    gpointer padding[22];
>  };

Although you maintained the struct size ABI, this is still technically
an API incompatible change / break.

FWIW this is equivalent the the same named slots in the domain class,
where they are used for signals. It seems we never wired up the signals
in the network class though, hence why they may appear uneccessary. So
I think we shouldn't remove them, but instead add the use by signals
as we have for other classes

Regards,
Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|




More information about the libvir-list mailing list