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

[Zeeshan Ali (Khattak)]

On Tue, Jun 26, 2012 at 12:16 PM, Christophe Fergeau
<cfergeau at redhat.com> wrote:
> On Tue, Jun 26, 2012 at 05:56:13AM +0300, Zeeshan Ali (Khattak) wrote:
>> From: "Zeeshan Ali (Khattak)" <zeeshanak at gnome.org>
>>
>> - gvir_config_capabilities_cpu_add_feature
>> - gvir_config_capabilities_cpu_set_topology
>> ---
>>  libvirt-gconfig/libvirt-gconfig-capabilities-cpu.c |   32 ++++++++++++++++++++
>>  libvirt-gconfig/libvirt-gconfig-capabilities-cpu.h |    6 ++++
>>  libvirt-gconfig/libvirt-gconfig.sym                |    6 ++++
>>  3 files changed, 44 insertions(+)
>>
>> diff --git a/libvirt-gconfig/libvirt-gconfig-capabilities-cpu.c b/libvirt-gconfig/libvirt-gconfig-capabilities-cpu.c
>> index df77364..e517a20 100644
>> --- a/libvirt-gconfig/libvirt-gconfig-capabilities-cpu.c
>> +++ b/libvirt-gconfig/libvirt-gconfig-capabilities-cpu.c
>> @@ -56,6 +56,21 @@ gvir_config_capabilities_cpu_get_arch(GVirConfigCapabilitiesCpu *cpu)
>>      return gvir_config_object_get_node_content(GVIR_CONFIG_OBJECT(cpu), "arch");
>>  }
>>
>> +/**
>> + * gvir_config_capabilities_cpu_add_feature:
>> + *
>> + * Adds a new feature to CPU.
>
> I don't know if it's worth mentioning in the doc that some of these objects
> can be read-only? I'm not sure how to say that in a non-confusing way
> though.

Why are they read-only? You can change the XML in every way you like,
just that in case of capabilities itself, you cant do anything with
the modified XML but since we don't have any API for setting caps, its
already obvious?

-- 
Regards,

Zeeshan Ali (Khattak)
FSF member#5124