diff options
| author | Simon McVittie <simon.mcvittie@collabora.co.uk> | 2014-01-27 17:25:55 +0000 |
|---|---|---|
| committer | Simon McVittie <simon.mcvittie@collabora.co.uk> | 2014-01-29 19:28:33 +0000 |
| commit | 37221c0f0955fe6afa33eba483d45e5204700c35 (patch) | |
| tree | 4132700138dab154e309f3968ffc7e84e26e9cb2 | |
| parent | c0b9663101092257873410d92e2c02219d109353 (diff) | |
_mcd_account_manager_setup: simplify a loop
As per Xavier's review, we don't need to call
mcd_account_get_unique_name(), because account unique names (object path
tails) are the keys in this map anyway.
Bug: https://bugs.freedesktop.org/show_bug.cgi?id=27727
| -rw-r--r-- | src/mcd-account-manager.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/src/mcd-account-manager.c b/src/mcd-account-manager.c index fa031161..5ff7bc9c 100644 --- a/src/mcd-account-manager.c +++ b/src/mcd-account-manager.c @@ -1510,10 +1510,9 @@ _mcd_account_manager_setup (McdAccountManager *account_manager) * It was added to uncork_storage_plugins() in 3d5b5e7a248d * without explanation */ g_hash_table_iter_init (&iter, account_manager->priv->accounts); - while (g_hash_table_iter_next (&iter, NULL, &v)) + while (g_hash_table_iter_next (&iter, &k, NULL)) { - mcd_storage_commit (storage, - mcd_account_get_unique_name (v)); + mcd_storage_commit (storage, k); } /* uncork signals from storage plugins */ |