[gnome-photos/wip/rishi/manager-model: 1/5] base-manager: Stop using photos_base_manager_get_objects
- From: Debarshi Ray <debarshir src gnome org>
- To: commits-list gnome org
- Cc:
- Subject: [gnome-photos/wip/rishi/manager-model: 1/5] base-manager: Stop using photos_base_manager_get_objects
- Date: Sat, 5 Nov 2016 17:55:31 +0000 (UTC)
commit d57658a4dad0f772730a0e1e787938df8984dc40
Author: Debarshi Ray <debarshir gnome org>
Date: Sat Nov 5 17:07:25 2016 +0100
base-manager: Stop using photos_base_manager_get_objects
The photos_base_manager_get_objects method is headed for removal. It is
not a good idea to publicly expose the BaseManager's GHashTable. Once
we implement the GListModel interface, the GHashTable will have a lot
more internal data and won't be a simple map from ID to GObject.
src/photos-base-manager.c | 8 ++++----
1 files changed, 4 insertions(+), 4 deletions(-)
---
diff --git a/src/photos-base-manager.c b/src/photos-base-manager.c
index 26d89b3..190cbc2 100644
--- a/src/photos-base-manager.c
+++ b/src/photos-base-manager.c
@@ -484,16 +484,16 @@ photos_base_manager_get_where (PhotosBaseManager *self, gint flags)
void
photos_base_manager_process_new_objects (PhotosBaseManager *self, GHashTable *new_objects)
{
- GHashTable *old_objects;
+ PhotosBaseManagerPrivate *priv;
GHashTableIter iter;
GObject *object;
const gchar *id;
g_return_if_fail (PHOTOS_IS_BASE_MANAGER (self));
- old_objects = photos_base_manager_get_objects (self);
+ priv = photos_base_manager_get_instance_private (self);
- g_hash_table_iter_init (&iter, old_objects);
+ g_hash_table_iter_init (&iter, priv->objects);
while (g_hash_table_iter_next (&iter, (gpointer *) &id, (gpointer *) &object))
{
gboolean builtin;
@@ -517,7 +517,7 @@ photos_base_manager_process_new_objects (PhotosBaseManager *self, GHashTable *ne
/* If new items are not found in the older hash table, add
* them.
*/
- if (g_hash_table_lookup (old_objects, id) == NULL)
+ if (g_hash_table_lookup (priv->objects, id) == NULL)
photos_base_manager_add_object (self, object);
}
[
Date Prev][
Date Next] [
Thread Prev][
Thread Next]
[
Thread Index]
[
Date Index]
[
Author Index]