diff options
| author | Xavier Claessens <xclaesse@gmail.com> | 2011-11-16 16:07:25 +0100 |
|---|---|---|
| committer | Xavier Claessens <xclaesse@gmail.com> | 2011-11-16 16:07:25 +0100 |
| commit | ee2090321ca1530ada26c5431b9ba3dacfd4ea07 (patch) | |
| tree | db9a88fe1e3b7e983c3ee08e13b50598931e8d2f /src/mcd-account-manager.c | |
| parent | 3e39bb0e14bcbbefcb98839e730d86864165f28b (diff) | |
Use _unref instead of _free _destroy when possible.unref
Replace g_(ptr_)array_free (foo, TRUE) and g_hash_table_destroy
with respectively g_(ptr_)array_unref (foo) and g_hash_table_unref.
I used this command to generate this patch:
for f in `find -name "*.c"`; do sed -i $f -re 's/g_ptr_array_free \(([^ ,]+), TRUE\)/g_ptr_array_unref \(\1\)/'; done
See Danielle's blog for explanation of possible bug _free can do:
http://blogs.gnome.org/danni/2011/11/16/mistakes-with-g_value_set_boxed/
Diffstat (limited to 'src/mcd-account-manager.c')
| -rw-r--r-- | src/mcd-account-manager.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/mcd-account-manager.c b/src/mcd-account-manager.c index c49a8e75..7714c137 100644 --- a/src/mcd-account-manager.c +++ b/src/mcd-account-manager.c @@ -1298,7 +1298,7 @@ _mcd_account_manager_finalize (GObject *object) remove (priv->account_connections_file); g_free (priv->account_connections_file); - g_hash_table_destroy (priv->accounts); + g_hash_table_unref (priv->accounts); G_OBJECT_CLASS (mcd_account_manager_parent_class)->finalize (object); } |