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
pgpFz86mQLZWO.pgp
Description: PGP signature
-- libvir-list mailing list [email protected] https://www.redhat.com/mailman/listinfo/libvir-list
