[gnome-photos/wip/rishi/manager-model: 3/5] share-point-manager: Rearrange the code for subsequent changes

[Debarshi Ray <debarshir src gnome org>]

commit 1630908286b3f354ec32f72f13c72b57e27c0752
Author: Debarshi Ray <debarshir gnome org>
Date:   Sat Nov 12 16:23:35 2016 +0100

    share-point-manager: Rearrange the code for subsequent changes
    
    https://bugzilla.gnome.org/show_bug.cgi?id=774191

 src/photos-share-point-manager.c |   22 +++++++++++-----------
 1 files changed, 11 insertions(+), 11 deletions(-)
---
diff --git a/src/photos-share-point-manager.c b/src/photos-share-point-manager.c
index 0a351db..47a9271 100644
--- a/src/photos-share-point-manager.c
+++ b/src/photos-share-point-manager.c
@@ -133,16 +133,18 @@ photos_share_point_manager_refresh_share_points (PhotosSharePointManager *self)
   g_hash_table_iter_init (&iter, sources);
   while (g_hash_table_iter_next (&iter, NULL, (gpointer *) &source))
     {
-      PhotosSharePoint *share_point;
-      const gchar *id;
+      PhotosSharePoint *share_point = NULL;
 
       share_point = photos_share_point_manager_create_share_point_online (self, source);
-      if (share_point == NULL)
-        continue;
+      if (share_point != NULL)
+        {
+          const gchar *id;
 
-      id = photos_filterable_get_id (PHOTOS_FILTERABLE (share_point));
-      g_hash_table_insert (new_share_points, g_strdup (id), g_object_ref (share_point));
-      g_object_unref (share_point);
+          id = photos_filterable_get_id (PHOTOS_FILTERABLE (share_point));
+          g_hash_table_insert (new_share_points, g_strdup (id), g_object_ref (share_point));
+        }
+
+      g_clear_object (&share_point);
     }
 
   photos_base_manager_process_new_objects (PHOTOS_BASE_MANAGER (self), new_share_points);
@@ -273,10 +275,8 @@ photos_share_point_manager_get_for_item (PhotosSharePointManager *self, PhotosBa
   g_hash_table_iter_init (&iter, share_points);
   while (g_hash_table_iter_next (&iter, (gpointer *) &share_point_id, (gpointer *) &share_point))
     {
-      if (g_strcmp0 (resource_urn, share_point_id) == 0)
-        continue;
-
-      ret_val = g_list_prepend (ret_val, g_object_ref (share_point));
+      if (g_strcmp0 (resource_urn, share_point_id) != 0)
+        ret_val = g_list_prepend (ret_val, g_object_ref (share_point));
     }
 
  out: