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

Re: [libvirt] [libvirt-glib 1/8] Setters/adders for GVirConfigCapabilitiesCpu props



On Mon, Jun 25, 2012 at 01:34:41PM +0200, Christophe Fergeau wrote:
> On Sat, Jun 23, 2012 at 10:17:57PM +0300, Zeeshan Ali (Khattak) wrote:
> > From: "Zeeshan Ali (Khattak)" <zeeshanak gnome org>
> > 
> > - gvir_config_capabilities_cpu_add_feature
> > - gvir_config_capabilities_cpu_set_topology
> 
> As I understand it, the capabilities XML is read-only, the only reason for
> these setters to exist is for use with GVirConfigDomainCpu which is added
> later in this patch set. I think I'd put the setters in this new class even
> if this makes the API asymetric (getters in one class, setters in another).

Yes & no. Technically you are correct that the capabilities XML is
readonly, however, when using the test driver in libvirt, it is
possible to pass a custom XML document fully describing the capabilities
that the test driver should return. Thus it is useful to have setters
in the GVirConfigCapabilities  class

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 :|


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