[libvirt] [libvirt-glib 29/37] Add GVirConfigDeviceInput

[Christophe Fergeau]

On Fri, Nov 11, 2011 at 08:32:30PM +0100, Marc-André Lureau wrote:
> just a comment about naming.
> 
> If we use prefix "Device" before devices class, I think it should be
> done for the rest of the class for consistency.

What are you referring to? GVirConfigGraphicsSpice and GVirConfigInterface?

> > +void gvir_config_device_input_set_bus(GVirConfigDeviceInput *input,
> > +                                      const char *bus)
> > +{
> > +    xmlNodePtr node;
> > +
> > +    node = gvir_config_object_get_xml_node(GVIR_CONFIG_OBJECT(input));
> > +    if (node == NULL)
> > +        return;
> > +    xmlNewProp(node, (xmlChar*)"bus", (xmlChar*)bus);
> > +}
> 
> The bus argument could be an enum too ("xen" (paravirtualized), "ps2" and "usb")

Ok.

Christophe
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20111115/6fad112b/attachment-0001.sig>