[Pkg-telepathy-commits] [telepathy-mission-control-6] 225/280: _mcd_account_manager_setup: simplify a loop

Simon McVittie smcv at debian.org
Thu Mar 27 20:07:27 UTC 2014


This is an automated email from the git hooks/post-receive script.

smcv pushed a commit to branch debian
in repository telepathy-mission-control-6.

commit 37221c0f0955fe6afa33eba483d45e5204700c35
Author: Simon McVittie <simon.mcvittie at collabora.co.uk>
Date:   Mon Jan 27 17:25:55 2014 +0000

    _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
---
 src/mcd-account-manager.c | 5 ++---
 1 file changed, 2 insertions(+), 3 deletions(-)

diff --git a/src/mcd-account-manager.c b/src/mcd-account-manager.c
index fa03116..5ff7bc9 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 */

-- 
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-telepathy/telepathy-mission-control-6.git



More information about the Pkg-telepathy-commits mailing list