Use of 'const' parameter on object parameters is redundant, inconsistent (both internally and against other GObject libraries) and currently breaks the low-level Rust binding generator: https://github.com/gtk-rs/glib/pull/249 --- osinfo/osinfo_install_config_param.c | 10 +++++----- osinfo/osinfo_install_config_param.h | 10 +++++----- osinfo/osinfo_install_script.c | 10 +++++----- osinfo/osinfo_install_script.h | 10 +++++----- 4 files changed, 20 insertions(+), 20 deletions(-) diff --git a/osinfo/osinfo_install_config_param.c b/osinfo/osinfo_install_config_param.c index f74a802..4adcd07 100644 --- a/osinfo/osinfo_install_config_param.c +++ b/osinfo/osinfo_install_config_param.c @@ -221,7 +221,7 @@ OsinfoInstallConfigParam *osinfo_install_config_param_new(const gchar *name) * * Returns: (transfer none): the name of the configuration parameter */ -const gchar *osinfo_install_config_param_get_name(const OsinfoInstallConfigParam *config_param) +const gchar *osinfo_install_config_param_get_name(OsinfoInstallConfigParam *config_param) { return osinfo_entity_get_param_value(OSINFO_ENTITY(config_param), OSINFO_INSTALL_CONFIG_PARAM_PROP_NAME); @@ -233,7 +233,7 @@ const gchar *osinfo_install_config_param_get_name(const OsinfoInstallConfigParam * * Returns: (transfer none): the policy of the configuration parameter */ -OsinfoInstallConfigParamPolicy osinfo_install_config_param_get_policy(const OsinfoInstallConfigParam *config_param) +OsinfoInstallConfigParamPolicy osinfo_install_config_param_get_policy(OsinfoInstallConfigParam *config_param) { return osinfo_entity_get_param_value_enum(OSINFO_ENTITY(config_param), OSINFO_INSTALL_CONFIG_PARAM_PROP_POLICY, @@ -248,7 +248,7 @@ OsinfoInstallConfigParamPolicy osinfo_install_config_param_get_policy(const Osin * Returns: (transfer none): TRUE if the config_param is required. * FALSE otherwise. */ -gboolean osinfo_install_config_param_is_required(const OsinfoInstallConfigParam *config_param) +gboolean osinfo_install_config_param_is_required(OsinfoInstallConfigParam *config_param) { return (osinfo_install_config_param_get_policy(config_param) == OSINFO_INSTALL_CONFIG_PARAM_POLICY_REQUIRED); @@ -261,13 +261,13 @@ gboolean osinfo_install_config_param_is_required(const OsinfoInstallConfigParam * Returns: (transfer none): TRUE if the config_param is optional. * FALSE otherwise. */ -gboolean osinfo_install_config_param_is_optional(const OsinfoInstallConfigParam *config_param) +gboolean osinfo_install_config_param_is_optional(OsinfoInstallConfigParam *config_param) { return (osinfo_install_config_param_get_policy(config_param) == OSINFO_INSTALL_CONFIG_PARAM_POLICY_OPTIONAL); } -OsinfoDatamap *osinfo_install_config_param_get_value_map(const OsinfoInstallConfigParam *config_param) +OsinfoDatamap *osinfo_install_config_param_get_value_map(OsinfoInstallConfigParam *config_param) { return config_param->priv->value_map; } diff --git a/osinfo/osinfo_install_config_param.h b/osinfo/osinfo_install_config_param.h index 58bb40b..8c083f3 100644 --- a/osinfo/osinfo_install_config_param.h +++ b/osinfo/osinfo_install_config_param.h @@ -81,16 +81,16 @@ GType osinfo_install_config_param_get_type(void); OsinfoInstallConfigParam *osinfo_install_config_param_new(const gchar *name); -const gchar *osinfo_install_config_param_get_name(const OsinfoInstallConfigParam *config_param); +const gchar *osinfo_install_config_param_get_name(OsinfoInstallConfigParam *config_param); -OsinfoInstallConfigParamPolicy osinfo_install_config_param_get_policy(const OsinfoInstallConfigParam *config_param); +OsinfoInstallConfigParamPolicy osinfo_install_config_param_get_policy(OsinfoInstallConfigParam *config_param); -gboolean osinfo_install_config_param_is_required(const OsinfoInstallConfigParam *config_param); +gboolean osinfo_install_config_param_is_required(OsinfoInstallConfigParam *config_param); -gboolean osinfo_install_config_param_is_optional(const OsinfoInstallConfigParam *config_param); +gboolean osinfo_install_config_param_is_optional(OsinfoInstallConfigParam *config_param); void osinfo_install_config_param_set_value_map(OsinfoInstallConfigParam *config_param, OsinfoDatamap *datamap); -OsinfoDatamap *osinfo_install_config_param_get_value_map(const OsinfoInstallConfigParam *config_param); +OsinfoDatamap *osinfo_install_config_param_get_value_map(OsinfoInstallConfigParam *config_param); #endif /* __OSINFO_INSTALL_CONFIG_PARAM_H__ */ diff --git a/osinfo/osinfo_install_script.c b/osinfo/osinfo_install_script.c index 4856eae..52b6d75 100644 --- a/osinfo/osinfo_install_script.c +++ b/osinfo/osinfo_install_script.c @@ -266,7 +266,7 @@ void osinfo_install_script_add_config_param(OsinfoInstallScript *script, OsinfoI OSINFO_ENTITY(param)); } -gboolean osinfo_install_script_has_config_param(const OsinfoInstallScript *script, const OsinfoInstallConfigParam *config_param) +gboolean osinfo_install_script_has_config_param(OsinfoInstallScript *script, OsinfoInstallConfigParam *config_param) { /* NB: this code assumes that the 'id' and 'name' entity properties * are the same @@ -275,7 +275,7 @@ gboolean osinfo_install_script_has_config_param(const OsinfoInstallScript *scrip return osinfo_install_script_has_config_param_name(script, name); } -gboolean osinfo_install_script_has_config_param_name(const OsinfoInstallScript *script, const gchar *name) +gboolean osinfo_install_script_has_config_param_name(OsinfoInstallScript *script, const gchar *name) { OsinfoList *l = OSINFO_LIST(script->priv->config_params); return (osinfo_list_find_by_id(l, name) != NULL); @@ -291,7 +291,7 @@ gboolean osinfo_install_script_has_config_param_name(const OsinfoInstallScript * * list of valid #OsinfoInstallConfigParam parameters. Free with * g_list_free() when done. The elements are owned by libosinfo. */ -GList *osinfo_install_script_get_config_param_list(const OsinfoInstallScript *script) +GList *osinfo_install_script_get_config_param_list(OsinfoInstallScript *script) { return osinfo_list_get_elements(OSINFO_LIST(script->priv->config_params)); } @@ -305,7 +305,7 @@ GList *osinfo_install_script_get_config_param_list(const OsinfoInstallScript *sc * Returns: (transfer none): the list of valid #OsinfoInstallConfigParam * parameters. */ -OsinfoInstallConfigParamList *osinfo_install_script_get_config_params(const OsinfoInstallScript *script) +OsinfoInstallConfigParamList *osinfo_install_script_get_config_params(OsinfoInstallScript *script) { return script->priv->config_params; } @@ -321,7 +321,7 @@ OsinfoInstallConfigParamList *osinfo_install_script_get_config_params(const Osin * NULL otherwise. */ OsinfoInstallConfigParam * -osinfo_install_script_get_config_param(const OsinfoInstallScript *script, +osinfo_install_script_get_config_param(OsinfoInstallScript *script, const gchar *name) { /* NB: this code assumes that the 'id' and 'name' entity properties diff --git a/osinfo/osinfo_install_script.h b/osinfo/osinfo_install_script.h index 61bf0e4..e07ab5e 100644 --- a/osinfo/osinfo_install_script.h +++ b/osinfo/osinfo_install_script.h @@ -235,14 +235,14 @@ gchar *osinfo_install_script_generate_command_line_for_media(OsinfoInstallScript OsinfoMedia *media, OsinfoInstallConfig *config); -gboolean osinfo_install_script_has_config_param(const OsinfoInstallScript *script, const OsinfoInstallConfigParam *config_param); +gboolean osinfo_install_script_has_config_param(OsinfoInstallScript *script, OsinfoInstallConfigParam *config_param); -gboolean osinfo_install_script_has_config_param_name(const OsinfoInstallScript *script, const gchar *name); +gboolean osinfo_install_script_has_config_param_name(OsinfoInstallScript *script, const gchar *name); -OsinfoInstallConfigParam *osinfo_install_script_get_config_param(const OsinfoInstallScript *script, const gchar *name); +OsinfoInstallConfigParam *osinfo_install_script_get_config_param(OsinfoInstallScript *script, const gchar *name); -GList *osinfo_install_script_get_config_param_list(const OsinfoInstallScript *script); -OsinfoInstallConfigParamList *osinfo_install_script_get_config_params(const OsinfoInstallScript *script); +GList *osinfo_install_script_get_config_param_list(OsinfoInstallScript *script); +OsinfoInstallConfigParamList *osinfo_install_script_get_config_params(OsinfoInstallScript *script); OsinfoPathFormat osinfo_install_script_get_path_format(OsinfoInstallScript *script); gboolean osinfo_install_script_get_can_pre_install_drivers(OsinfoInstallScript *script); -- 2.13.6 _______________________________________________ Libosinfo mailing list Libosinfo@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libosinfo