diff options
| author | Xavier Claessens <xclaesse@gmail.com> | 2011-11-16 14:08:45 +0100 |
|---|---|---|
| committer | Xavier Claessens <xclaesse@gmail.com> | 2011-11-16 14:36:51 +0100 |
| commit | 67bffca0678d1286d07b1531f6562126e92cc31e (patch) | |
| tree | 309fa93328b63d8c1a90432bc1c697b4a97beba0 /examples/client | |
| parent | e6d8b4c7531688cd2dbd3c9bbd8a6c224123de05 (diff) | |
Use _unref instead of _free _destroy when possible.
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 'examples/client')
| -rw-r--r-- | examples/client/extended-client.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/examples/client/extended-client.c b/examples/client/extended-client.c index bf01b8777..678cbc6dc 100644 --- a/examples/client/extended-client.c +++ b/examples/client/extended-client.c @@ -162,7 +162,7 @@ contacts_ready_cb (TpConnection *conn, "red", EXAMPLE_HAT_STYLE_FEDORA, asv, set_hat_cb, pair, contact_pair_free, NULL); - g_hash_table_destroy (asv); + g_hash_table_unref (asv); } static void @@ -272,7 +272,7 @@ connection_manager_got_info (TpConnectionManager *cm, tp_cli_connection_manager_call_request_connection (cm, -1, "example", params, cm_requested_connection, NULL, NULL, NULL); - g_hash_table_destroy (params); + g_hash_table_unref (params); } } |