[empathy/mc5] Fix some leaks in _settings_discard
- From: Sjoerd Simons <sjoerds src gnome org>
- To: svn-commits-list gnome org
- Cc:
- Subject: [empathy/mc5] Fix some leaks in _settings_discard
- Date: Sat, 22 Aug 2009 18:55:09 +0000 (UTC)
commit c10e3fbca90c684a3f0988a119cfad3d14def218
Author: Sjoerd Simons <sjoerd simons collabora co uk>
Date: Sat Aug 22 17:44:14 2009 +0100
Fix some leaks in _settings_discard
Prevent leaking the unset parameter names when unsetting the array.
Also use it as a general utility function in other parts of the code
libempathy/empathy-account-settings.c | 26 +++++++++++++-------------
1 files changed, 13 insertions(+), 13 deletions(-)
---
diff --git a/libempathy/empathy-account-settings.c b/libempathy/empathy-account-settings.c
index f6c1af4..efcbb2d 100644
--- a/libempathy/empathy-account-settings.c
+++ b/libempathy/empathy-account-settings.c
@@ -630,14 +630,8 @@ empathy_account_settings_discard_changes (EmpathyAccountSettings *settings)
{
EmpathyAccountSettingsPriv *priv = GET_PRIV (settings);
- if (g_hash_table_size (priv->parameters) > 0)
- g_hash_table_remove_all (priv->parameters);
-
- if (priv->unset_parameters->len > 0)
- {
- g_array_remove_range (priv->unset_parameters, 0,
- priv->unset_parameters->len);
- }
+ g_hash_table_remove_all (priv->parameters);
+ empathy_account_settings_free_unset_parameters (settings);
}
const gchar *
@@ -967,6 +961,10 @@ empathy_account_settings_account_updated (GObject *source,
g_simple_async_result_set_from_error (priv->apply_result, error);
g_error_free (error);
}
+ else
+ {
+ empathy_account_settings_discard_changes (settings);
+ }
r = priv->apply_result;
priv->apply_result = NULL;
@@ -990,9 +988,14 @@ empathy_account_settings_created_cb (GObject *source,
EMPATHY_ACCOUNT_MANAGER (source), result, &error);
if (account == NULL)
- g_simple_async_result_set_from_error (priv->apply_result, error);
+ {
+ g_simple_async_result_set_from_error (priv->apply_result, error);
+ }
else
- priv->account = g_object_ref (account);
+ {
+ priv->account = g_object_ref (account);
+ empathy_account_settings_discard_changes (settings);
+ }
r = priv->apply_result;
priv->apply_result = NULL;
@@ -1111,9 +1114,6 @@ empathy_account_settings_apply_async (EmpathyAccountSettings *settings,
empathy_account_update_settings_async (priv->account,
priv->parameters, (const gchar **)priv->unset_parameters->data,
empathy_account_settings_account_updated, settings);
-
- g_hash_table_remove_all (priv->parameters);
- empathy_account_settings_free_unset_parameters (settings);
}
}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]