From: "Zeeshan Ali (Khattak)" <[email protected]>
Although this means ABI "breakage", we must keep in mind:
a. The property getter (the API used by apps) was treating the string
value as enum, i-e it was broken for apps so if any app is using this
API, they were totally screwed anyways.
b. Its doubtful that this new API is used by any application out there.
---
osinfo/osinfo_install_config_param.c | 24 +++++++++++++-----------
1 file changed, 13 insertions(+), 11 deletions(-)
diff --git a/osinfo/osinfo_install_config_param.c
b/osinfo/osinfo_install_config_param.c
index 6d65c9b..0c8b96a 100644
--- a/osinfo/osinfo_install_config_param.c
+++ b/osinfo/osinfo_install_config_param.c
@@ -155,15 +155,15 @@ osinfo_install_config_param_class_init
(OsinfoInstallConfigParamClass *klass)
*
* The policy of the configuration parameter.
**/
- pspec = g_param_spec_string("policy",
- "Policy",
- _("Parameter policy"),
- NULL,
- G_PARAM_READABLE |
- G_PARAM_CONSTRUCT_ONLY |
- G_PARAM_STATIC_NAME |
- G_PARAM_STATIC_NICK |
- G_PARAM_STATIC_BLURB);
+ pspec = g_param_spec_enum("policy",
+ "Policy",
+ _("Parameter policy"),
+ OSINFO_TYPE_INSTALL_CONFIG_PARAM_POLICY,
+ OSINFO_INSTALL_CONFIG_PARAM_POLICY_OPTIONAL,
+ G_PARAM_READABLE |
+ G_PARAM_STATIC_NAME |
+ G_PARAM_STATIC_NICK |
+ G_PARAM_STATIC_BLURB);
g_object_class_install_property(g_klass,
PROP_POLICY,
pspec);
@@ -218,8 +218,10 @@ const gchar *osinfo_install_config_param_get_name(const
OsinfoInstallConfigParam
*/
OsinfoInstallConfigParamPolicy osinfo_install_config_param_get_policy(const
OsinfoInstallConfigParam *config_param)
{
- return osinfo_entity_get_param_value(OSINFO_ENTITY(config_param),
-
OSINFO_INSTALL_CONFIG_PARAM_PROP_POLICY);
+ return osinfo_entity_get_param_value_enum(OSINFO_ENTITY(config_param),
+
OSINFO_INSTALL_CONFIG_PARAM_PROP_POLICY,
+
OSINFO_TYPE_INSTALL_CONFIG_PARAM_POLICY,
+
OSINFO_INSTALL_CONFIG_PARAM_POLICY_OPTIONAL);
}
/**
--
1.8.0
_______________________________________________
virt-tools-list mailing list
[email protected]
https://www.redhat.com/mailman/listinfo/virt-tools-list
