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

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



On Tue, Jun 26, 2012 at 12:16 PM, Christophe Fergeau
<cfergeau redhat com> wrote:
> On Tue, Jun 26, 2012 at 05:56:13AM +0300, Zeeshan Ali (Khattak) wrote:
>> From: "Zeeshan Ali (Khattak)" <zeeshanak 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


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