Re: [libvirt-glib 30/37] Add basic GVirConfigGraphicsSpice

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Same comment as for "interface", do we want to have specialized
classes for various type.

In this case, you should also add a base Graphics class.

> +GVirConfigGraphicsSpice *gvir_config_graphics_spice_new_from_xml(const gchar *xml,
> +                                                                 GError **error)
> +{
> +    GVirConfigObject *object;
> +
> +    object = gvir_config_object_new_from_xml(GVIR_TYPE_CONFIG_GRAPHICS_SPICE,
> +                                             "graphics", NULL, xml, error);
> +//    xmlNewProp(doc->children, (xmlChar*)"type", (xmlChar*)"spice");
> +    return GVIR_CONFIG_GRAPHICS_SPICE(object);
> +}

This surely shouldn't be commented out.
Perhaps it should be done in _init().

-- 
Marc-André Lureau

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]